HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...
Bug #857724
openMid-turn Governor (CMA) changes not saved when game is saved manually
0%
Description
I set my Governor setting for my cities and took a screenshot (attached). I saved the game (attached). I quit freeciv. Start freeciv again, load the savegame and look at the Governor settings and see that they have changed. Screenshots attached.
Files
Updated by Jacob Nevins almost 5 years ago
- Governor presets in the city report are matched by value (that is, each city stores only numeric settings, and the client tries to match these to a named preset for display). If there are multiple presets with the same values, perhaps which one wins can change on a fresh client start. (I can see that in your savegame, two of your cities match the default "Prefer production" preset and one matches "Prefer food", matching your 'after' screenshot.) (Arguably the bug is that multiple presets with the same values should be prevented.)
- Governor presets are saved in freeciv-client-rc (client settings) file, not the savegame. If you changed the presets, and client settings weren't saved before you quit (by you or automatically), then your preset changes will be lost. (I don't think there's a bug here, since the default is to save automatically on quit.)
- Finally, quitting mid-turn is known to lose any changes to cities' governor settings, due to the way the governor works. See gna bug #18850 . That's a (design) bug. I can't tell if it bit you. (I thought it was documented, but I don't see it in online help or README.governor.)
Updated by Chippo Elder almost 5 years ago
- File freeciv-client-rc-2.6 freeciv-client-rc-2.6 added
Jacob Nevins wrote:
See gna bug #18850 . That's a (design) bug. I can't tell if it bit you.
That seems to be the bug. So, we can reject this ticket, I think.
I attached my client settings just in case.
Updated by Marko Lindqvist almost 5 years ago
- Sprint/Milestone set to 3.1.0
We had the gna bug, but I don't think we have had hrm ticket before this.
Current design had been chosen on the assumption that there would be more users for the general client attributes design. There never has been second user for that. We can keep the attributes design around, but we should change governor settings to not use them. The data they have has been unchanged for a very long time, so it's unlikely that there would be major compatibility problems with dedicated design (network protocol, savefile format) over versions.
Updated by Marko Lindqvist almost 5 years ago
- Blocks Task #673656: S3_1 datafile format freeze (d3f) added
Updated by Jacob Nevins almost 5 years ago
- Subject changed from Save/load cycle resets the Governor settings (2.6.1+) to Mid-turn Governor (CMA) changes not saved when game is saved manually
Updated by Chippo Elder almost 5 years ago
I had to search around (very little) to find out what a CMA was. If CMA is preferred over Governor, shouldn't we change the guis to say CMA instead of Governor in that Cities column and in the City dialog?
Updated by Marko Lindqvist almost 5 years ago
Chippo Elder wrote:
I had to search around (very little) to find out what a CMA was. If CMA is preferred over Governor, shouldn't we change the guis to say CMA instead of Governor in that Cities column and in the City dialog?
Governor is a (more user-friendly) synonym for CMA.
Updated by Jacob Nevins almost 5 years ago
shouldn't we change the guis to say CMA
We went the other way a mere 13 years ago...
(I just put "CMA" in the subject to match the Gna ticket, and make clear to people reading code that this is about that specific bit of Freeciv, not other bits that people think of as the "governor" like what the code calls the building advisor. Wasn't meant as a criticism, "Governor" is fine.)
Updated by Marko Lindqvist over 2 years ago
Marko Lindqvist wrote:
We can keep the attributes design around, but we should change governor settings to not use them.
That's certainly too big a redesign to enter S3_1 any more. So the overall ticket is going to get postponed. I am, however, still trying to find a way to somehow mitigate the problem even in 3.1.
Updated by Marko Lindqvist over 2 years ago
- Sprint/Milestone changed from 3.1.0 to 3.2.0
Marko Lindqvist wrote:
So the overall ticket is going to get postponed.
Doing that now.
still trying to find a way to somehow mitigate the problem even in 3.1.
Mitigation in https://osdn.net/projects/freeciv/ticket/44304 can actually be applied to even older branches.
Updated by Marko Lindqvist over 2 years ago
- Blocks Task #939772: S3_2 datafile format freeze (d3f) added
Updated by Marko Lindqvist over 2 years ago
- Blocks deleted (Task #673656: S3_1 datafile format freeze (d3f))
Updated by Marko Lindqvist 6 months ago
Can't see a solution any time soon. Dropping d3f blocking completely.
Updated by Marko Lindqvist 6 months ago
- Blocks deleted (Task #939772: S3_2 datafile format freeze (d3f))