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
Lifetime profit #7919
Lifetime profit #7919
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of these comments are just thoughts and opinions.
Before reading this code I kind of expected the lifetime profit to include the purchase cost of the vehicle/train. On the other hand, that might be difficult to keep track of with trains composed of multiple vehicles, i.e. what happens if you add or remove cars from a train, how does that update the lifetime profit? (I think that can be solved, but I'm not sure if it's worth solving.)
1c85e0c
to
9ee5102
Compare
I have a dilema - what should be the conversion approach for old savegames? Renewed vehicles reset lifetime profit to 0, but not last year profit. It remains whatever it is. It could be a real new vehicle, or a renewed vehicle. If it's a real new vehicle, then getting its last year profit as best guess for lifetime profit is correct, no matter its age. But if the vehicle was renewed, then its last year profit was inherited from its past vehicle, and using its last year profit to determine lifetime profit, falls into the wrong side, unless the vehicle is already 1 year old at the time of conversion. How should I do the lifetime conversion?
|
3e797d9
to
df1a225
Compare
Wouldn't it be better to make the window resizable horizontally too or to put a horizontal scrollbar at the bottom of the vehicle list? Instead of making wider the whole window by default. |
df1a225
to
3ffc56b
Compare
3ffc56b
to
fb09f55
Compare
fb09f55
to
cc6e49c
Compare
cc6e49c
to
9f08c55
Compare
There is now a lifetime profit value against vehicles which can be viewed alongside the profit this year and profit last year. burty - Stores lifetime profit (gets updated at end of year) burty - Displays on the vehicle info window where the other profits are shown. burty - Saves the lifetime profit in to the save game and loads it. burty - Sort by lifetime profit in vehicle lists. JGR - Display the current lifetime profit, instead of the lifetime profit at the start of the current year JGR - Reset overall lifetime profit to 0 when renewing vehicle. JGR - Sort by length was incorrectly available, and crashed for non ground vehicles. Samu - Displays on the vehicle group info panel where the other profits are shown. Samu - Displays on the vehicle list window where the other profits are shown. Samu - Add AI/GS functions to get lifetime profits of groups and vehicles.
9f08c55
to
ffac2d0
Compare
This is once again a Pull Request which is impossible for us to review. It contains thoughts, it contains experiments, it contains lines of code that do not appear to belong in this PR. I am so confused, I am not even willing to look at this anymore. Please Samu, start with small PRs, of like 1 function, 1 tiny change. Look how other people are doing it, and how much more smooth we can guide them through this process. This PR is all over the place, making it impossible to guide you in any way that will lead to a result. And this is not the first time we are telling you this .. in fact, your recent PRs show that you are not learning at all (and there is a year between those two!). So again, I am not against this idea, but we do not have the time to go through this implementation and ensure it does not contain any game-breaking side-effects. |
There is now a lifetime profit value against vehicles which can be viewed alongside the profit this year and profit last year.
burty - Stores lifetime profit (gets updated at end of year)
burty - Displays on the vehicle info window where the other profits are shown.
burty - Saves the lifetime profit in to the save game and loads it.
burty - Sort by lifetime profit in vehicle lists.
JGR - Display the current lifetime profit, instead of the lifetime profit at the start of the current year
JGR - Reset overall lifetime profit to 0 when renewing vehicle.
JGR - Sort by length was incorrectly available, and crashed for non ground vehicles.
Samu - Displays on the vehicle group info panel where the other profits are shown.
Samu - Displays on the vehicle list window where the other profits are shown.
Samu - Add AI/GS functions to get lifetime profits of groups and vehicles.