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.

Sign up for GitHub

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

Jump to bottom

Log the invest version at the end of execution #1529

Closed
davemfish opened this issue Feb 20, 2024 · 0 comments · Fixed by #1577
Closed

Log the invest version at the end of execution #1529

davemfish opened this issue Feb 20, 2024 · 0 comments · Fixed by #1577
Assignees
@davemfish
Labels
enhancement New feature or request
Milestone
3.14.2

Comments

@davemfish
Copy link
Contributor

It's always helpful to know which version people ran when they submit error reports. invest logs its version at the start of a model execution, but it's quite common for forum users to paste in only the traceback rather than attach their whole logfile. So I think it would be useful if the final log messages of the execution manager also included the invest version. For example, invest logs always end with these two messages, and we could add a third to print the version again.

...
02/20/2024 11:23:03 natcap.invest.utils INFO Elapsed time: 0.35s
02/20/2024 11:23:03 natcap.invest.utils INFO Execution finished
@davemfish davemfish added the enhancement New feature or request label Feb 20, 2024
@davemfish davemfish added this to the 3.14.2 milestone Feb 20, 2024
@davemfish davemfish self-assigned this May 7, 2024
davemfish added a commit to davemfish/invest that referenced this issue May 10, 2024
@davemfish
log the invest version string at the end of execution. natcap#1529
8d2192b
@davemfish davemfish mentioned this issue May 10, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davemfish davemfish

Labels
enhancement New feature or request
Projects
None yet
Milestone
3.14.2
Development

Successfully merging a pull request may close this issue.

log the invest version string at the end of execution davemfish/invest
1 participant
@davemfish