Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It would be nice if dice stats were restored from saved games #3478

Closed
asvitkine opened this issue Jun 24, 2018 · 14 comments
Closed

It would be nice if dice stats were restored from saved games #3478

asvitkine opened this issue Jun 24, 2018 · 14 comments

Comments

@asvitkine
Copy link
Contributor

It would be nice if dice stats were part of the save game.

Right now, if you have a long-running game that you play over multiple days (e.g. with a human opponent over the internet), the dice stats gets reset whenever you reload the game.

It would be nice if they persisted.

@beelee1
Copy link
Contributor

beelee1 commented Jun 25, 2018

my saves have the dice in history still. Those are local games so idk if that makes a difference

@Cernelius
Copy link
Contributor

I never understood that, either.

I'm largely uninterested about that feature (or at least I virtually never look at it) but it seems almost pointless if you can only look at a whole game dice stats playing it from start to finish in one go and cannot reference it thereafter, losing all stats as soon as you quit it.

@Cernelius
Copy link
Contributor

But I think that the stats themselves shouldn't persist (that would mean being recorded in the savegame), but the program giving the dice stats should be able to go reading the history when you open it, rather than collecting the data since you start playing (that I assume it is what it is doing now).

@asvitkine
Copy link
Contributor Author

asvitkine commented Jun 25, 2018 via email

@asvitkine asvitkine changed the title It would be nice if dice stats were part of the save game It would be nice if dice stats were restored from save gamed Jun 25, 2018
@asvitkine
Copy link
Contributor Author

I'll take a look at implementing.

@Cernelius
Copy link
Contributor

If reading dice stats take any considerable amount of time, meaning it may make slower starting a game (mostly with reference to sending the data to the other participants in the host) or joining it when in progress etc., better doing it upon requesting the stats, rather than upon loading the game. Never mind if it doesn't really impact (like less than 1 second even for large maps with a long history).

@asvitkine
Copy link
Contributor Author

asvitkine commented Jun 26, 2018

I don't think it should take any significant time. You can test it out yourself - see #3483.

@Cernelius
Copy link
Contributor

How does it work for someone playing with you in a game you are hosting? It is you that make the stats and send the finalized info to the other player or do you send him the dice history and he creates the stats?

@Cernelius
Copy link
Contributor

Unfortunately I don't play the very big maps, like a round 40 or World At War or Domination. That is what I was thinking.

@asvitkine
Copy link
Contributor Author

asvitkine commented Jun 26, 2018 via email

@Cernelius
Copy link
Contributor

Ok, right, but what I was asking was if you meant "raw data" (that you use to make the statistics) or the statistics themselves. I'm not a developer.

@asvitkine
Copy link
Contributor Author

asvitkine commented Jun 26, 2018 via email

@Cernelius
Copy link
Contributor

Ok what I understand is that each terminal rebuilds the statistics from the history, and nothing changes about the data sent over the internet between terminals from now. Correct?

@asvitkine
Copy link
Contributor Author

asvitkine commented Jun 26, 2018 via email

@asvitkine asvitkine changed the title It would be nice if dice stats were restored from save gamed It would be nice if dice stats were restored from saved games Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants