This is part of meta issue987.

We copied all relevant information to issue1009 and mark this duplicate here as resolved now.

And I agree this is a duplicate.

Mark as resolved after copying the relevant information to the other thread. Deleting an issue is intended as an admin command, and we'd normally only do it for things like spam.

I believe this is a duplicate of issue1009, do you agree? What's the usual procedure in this case? Mark one as resolved or delete it (after copying the relevant information to the other thread)?

Yes. In that case, we just have to move the cost to the heuristic.

Doesn't the first point obviate the second? At the time we reach the heuristic, there is no such thing as a landmark factory any more.

The cost information of a landmark is currently initialized only by LandmarkFactoryRpgSasp (as the minimal cost over all achievers) and is only used by the LM count heuristic; for all other landmark factories, this is initialized to a default value of 1.

In this issue, we'd like to

- move the cost from the landmark class to the heuristic
- make sure the cost is initialized identically for all landmark factories