ritual site search bug/feature request

[DrPraetorious]

If you're going to make the search math more complicated at all, it might as well generate a priority array of provinces each turn, and crawl through that list in descending order.
I'm just guessing about how this works and then writing in C to get what I mean accross.

Code:

---

for (prov = 0; prov < num_prov; prov++){

if (owners[prov] == currentplayer){



slevel = map.sitefrequency + sitefrequencyadjust(terrain[site]);



for (i = const_FIRE; i <= const_HOLY; i++){

 /* If a site has been searched for one type and nothing has been found, chances of seeing something of another type go up. http://www.stat.sc.edu/~west/javahtm...MakeaDeal.html

 Also Bayesian inference generally. This is not strictly the right adjustment. */

 slevel += currentsearch[currentplayer][prov][i] * 25 / 7;

}



for (i = const_FIRE; i <= const_HOLY; i++){

 spri[prov,i] = slevel * (4 - knownfeatures[i]);

 spri[prov,i] /= exp(2,currentsearch[currentplayer][prov][i]);

/* This assumes that half of the sites are level 1, a quarter are level 2, etc. */

 if (currentsearch[currentplayer][prov][i] >= 4){spri[prov,i] = 0;}

/* And of course no sites are above level 4. */

}



if (isForest(prov)){spri[prov,const_NATURE]*=1.2;}

if (isMountain(prov){spri[prov,const_EARTH]*=1.2; spri[prov,const_AIR]*=1.1;}

if (isWater(prov){spri[prov,const_WATER]*=1.2;}

if (isSwamp(prov){sprit[prov,const_DEATH]*=1.2; spri[prov,const_WATER]*=1.1; spri[prov,const_NATURE]*=1.1;}

if (isWaste(prov){sprit[prov,const_DEATH]*=1.1; spri[prov,const_FIRE]*=1.2;}

}

}

---

And then you sort the province #s according to their priority for each given search type. For Akashic Record or Voice of Tiamat, just some the priorities before sorting.

If someone wants to do a statistical analysis of Edi's site list and get real values... actually I'm probably the only person here who'd want to do that.

[thejeff]

That would be nice, but could you add in something to prioritize more easily defended provinces as well.
At least ones which are farther within your empire. Or castled?
It's frustrating to be expanding downwards on the map and have your searchers always searching the just taken provinces, that are most likely to be fought over again.

More seriously, while a more efficient gem gaining algorithm would be good, the current version seems broken. It searches provinces it shouldn't and ignores some that it should. Fixing that seems more urgent than improving the algorithm.


The current version also has another unrelated flaw. The next province is chosen based on those provinces held during the ritual magic phase of the turn. A province can be targeted and then lost in the battle phase. The search will still be cast on the next turn.

[DrPraetorious]

All ritual magic happens before battles, changing that is a non-starter. Site searching spells are a minor point to make them an exception to the ordinary order of turn resolution.

As far as I can tell, what it does now is: it crawls through the list in order until it finds a site that isn't being targeted and that has none-zip-zero search points in that magic path.

Now, on the one hand, you do *eventually* want to cast Haurspex on provinces that got only a nature-1 search. OTOH, it would be a major pain if the spell searched those provinces *first*. So if you're going to change the way it works at all, you should have a prioritization algorithm.

[Gandalf Parker]

Anything could be made more efficient but sometimes I think they cut back on all the possible efficient code for hosting time.

Its too bad though. I would rather that ALL of the efficient code was put in. I dont really care how long a hosting takes.

[thejeff]

You may be misunderstanding the problem. It's not that the province is searched and then taken on the turn the spell is cast. That's expected and can occur casting the spells manually.
It's that the automated caster can target a province that is then taken, leaving the caster with orders to cast on a province you no longer own. This can't be duplicated manually, since you can't select that province.

It could be fixed with a quick cleanup check after the battle phase to clear out any monthly orders no longer valid. (Or preferably re-target to a now valid province.)

Actually a better fix would be to handle all of next months monthly orders later in the sequence. This would let you use this months gem income on monthly spells.


As for the current order, it looks to me like you are right about the intent, except that it skips some provinces, often, but not always and not all, capitols.

[DrPraetorious]

Ah, you're right. I was 100% mistaken and thejeff makes a good point. An extra phase for autogenerated orders (after gem income) would probably be a good idea.