Issue1025

Title Remove cost from landmark class and initialize costs identically in all landmark factories
Priority wish Status resolved
Superseder Nosy List clemens, florian, jendrik, malte, salome, silvan, thomas
Assigned To Keywords
Optional summary
This is part of meta issue987.

Created on 2021-06-28.16:15:34 by thomas, last changed by salome.

Summary
This is part of meta issue987.
Messages
msg10724 (view) Author: salome Date: 2022-05-25.12:05:32
We copied all relevant information to issue1009 and mark this duplicate here as resolved now.
msg10651 (view) Author: malte Date: 2022-03-09.16:58:39
And I agree this is a duplicate.
msg10650 (view) Author: malte Date: 2022-03-09.16:58:21
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.
msg10648 (view) Author: clemens Date: 2022-03-09.15:23:28
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)?
msg10335 (view) Author: thomas Date: 2021-06-28.17:06:11
Yes. In that case, we just have to move the cost to the heuristic.
msg10334 (view) Author: malte Date: 2021-06-28.16:32:08
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.
msg10333 (view) Author: thomas Date: 2021-06-28.16:15:34
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
History
Date User Action Args
2022-05-25 12:05:32salomesetstatus: chatting -> resolved
messages: + msg10724
2022-03-09 16:58:39maltesetmessages: + msg10651
2022-03-09 16:58:21maltesetmessages: + msg10650
2022-03-09 15:23:28clemenssetmessages: + msg10648
2021-07-12 15:36:51floriansetnosy: + florian
2021-06-28 17:06:11thomassetmessages: + msg10335
2021-06-28 16:32:08maltesetmessages: + msg10334
2021-06-28 16:15:34thomascreate