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

feat(version): use class StudyVersion to handle versions #2156

Merged
merged 32 commits into from
Oct 2, 2024

Conversation

MartinBelthle
Copy link
Contributor

@MartinBelthle MartinBelthle commented Sep 24, 2024

This PR uses the class StudyVersion defined inside the antares-study-version package to handle study versions widely inside the code.

It allows users to import studies in v9.0.
It writes the version as 9.0 in the database (compared to 880 before)

@MartinBelthle MartinBelthle force-pushed the feat/use-antares-study-version-widely branch from 959388a to 619946a Compare September 25, 2024 12:10
antarest/study/model.py Outdated Show resolved Hide resolved
version_to_write = f"{version:ddd}"
else: # should be written as X.Y
version_to_write = f"{version:2d}"
study_info["antares"]["solver_version"] = version_to_write
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need an evolution in antares-launcher to handle this correctly

sylvlecl
sylvlecl previously approved these changes Oct 2, 2024
@MartinBelthle MartinBelthle requested a review from sylvlecl October 2, 2024 12:00
@MartinBelthle MartinBelthle merged commit 31acc52 into dev Oct 2, 2024
10 of 11 checks passed
@MartinBelthle MartinBelthle deleted the feat/use-antares-study-version-widely branch October 2, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants