-
Notifications
You must be signed in to change notification settings - Fork 19
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
[gt4py] mysterious versions #838
Comments
@abishekg7 can you unconfuse me? |
Basically, gt4py has not have any official releases since v1.0.1 and the gt4py team reserved the numbered tag names for official releases. What we had instead was these temporary tag names. But on the spack side we needed the numbered tag names to allow for easier version management, and hence what you see here. All this said, we made a tiny error choosing these internal spack version numbers. It should have been v1.0.1.1, v1.0.1.2 instead of 1.1.1 and 1.1.2. @jonasjucker and I were just discussing about changing these version names (to v1.0.1.2 etc) so that when gt4py actually has an 1.0.2 release it won't conflict with our naming. |
v1.0.1.1, etc makes sense. You could also name them v20230413 to align with the tags if you like. |
Fair. For now I'll do the version renaming in a spack PR. But I'll bring it up with the gt4py team re. how to handle these intermediate tags better. |
The mysterious version are my bad. |
Wait until icon4py uses more stable gt4py versions. |
The versions listed here
spack-c2sm/repos/c2sm/packages/py-gt4py/package.py
Lines 19 to 21 in b7e55db
are not released https://github.com/GridTools/gt4py/releases .
This looks like gt4py didn't release these versions to the public, but the package manager spack knows about them. That's odd, isn't it?
Also https://github.com/GridTools/gt4py knows these commits as icon4py_20230413, while spack refers to them as 1.1.2. Why the mismatch? Isn't this confusing?
The text was updated successfully, but these errors were encountered: