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

Record message_ix version in scenario data & GDX I/O #747

Closed
khaeru opened this issue Oct 19, 2023 · 0 comments · Fixed by #765
Closed

Record message_ix version in scenario data & GDX I/O #747

khaeru opened this issue Oct 19, 2023 · 0 comments · Fixed by #765
Assignees
Labels
enh New features & functionality
Milestone

Comments

@khaeru
Copy link
Member

khaeru commented Oct 19, 2023

The GDX file generated by ixmp.JDBCBackend has a parameter named MESSAGE_ix_version with values like {"major": 2, "minor": 0}. The original intent of this was to aid with debugging when inspecting/comparing GDX files.

This has not been updated for recent MESSAGE versions, for multiple reasons:

  • It is a "stack level violation", in other words in the wrong place: ixmp (or its backends) should be agnostic to whether they are being used by message_ix or not; similarly, logic relevant to message_ix should be actually in message_ix.
  • The team lacks a Java developer capable of updating the code and the JAR file in ixmp.

Raised by @volker-krey in the 2023-10-19 MESSAGEix meeting.

Fix

  • We can use the code underlying show_versions() to generate a string with more informative version information (like " 3.7.1.dev55+g2e0a9c49"), and store this in a parameter.
  • This can be updated in message_ix.Scenario.solve().
  • The name should be something other than "MESSAGE_ix_version" (perhaps just "version"?) in order to avoid a collision with the (fixed) behaviour of the Java code.
@khaeru khaeru added the enh New features & functionality label Oct 19, 2023
@khaeru khaeru added this to the 3.8 milestone Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enh New features & functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants