Skip to content

Comments

fdb: update eccodes#38

Open
nnellen wants to merge 9 commits intomainfrom
update-eccodes
Open

fdb: update eccodes#38
nnellen wants to merge 9 commits intomainfrom
update-eccodes

Conversation

@nnellen
Copy link
Collaborator

@nnellen nnellen commented Dec 16, 2025

No description provided.

@nnellen
Copy link
Collaborator Author

nnellen commented Dec 16, 2025

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Dec 16, 2025

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Dec 18, 2025

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 3, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

Test if script is working without impoerting eccodes.
@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

jq --arg view "$view" '.views[$view].env.values.list.GRIB_DEFINITION_PATH = [{"op":"prepend","value":["/user-environment/data/share/eccodes-cosmo-mars/definitions", "/user-environment/data/share/definitions.edzw-2.38.3-1","/user-environment/env/fdb/share/eccodes/definitions"]}]' store/meta/env.json > tmp.json && mv tmp.json store/meta/env.json
jq --arg view "$view" '.views[$view].env.values.list.PATH[0].value |= ["/user-environment/venvs/fdb/bin/"] + .' store/meta/env.json > tmp.json && mv tmp.json store/meta/env.json
jq --arg view "$view" '.views[$view].env.values.list.FDB_ENABLE_GRIBJUMP = [{"op":"set","value":"1"}]' store/meta/env.json > tmp.json && mv tmp.json store/meta/env.json
jq --arg view "$view" '.views[$view].env.values.list.ECCODES_DIR = [{"op":"set","value":["/user-environment/env/$view/share/eccodes"]}]' store/meta/env.json > tmp.json && mv tmp.json store/meta/env.json
Copy link
Collaborator

Choose a reason for hiding this comment

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

The variable should point to the view since findlib will search in a lib or lib64 sub path

Copy link
Collaborator Author

@nnellen nnellen Feb 13, 2026

Choose a reason for hiding this comment

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

What view? The spack view? @cfkanesan

Copy link
Collaborator

Choose a reason for hiding this comment

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

/user-environment/env/$view

$python_exe -m venv /user-environment/venvs/fdb
source /user-environment/venvs/fdb/bin/activate
/user-environment/bin/poetry install
poetry remove eccodeslib
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is currently not documented but it looks like the correct way to install without the binary is to set the env var DEPEND_ON_ECCODESLIB to 0 before calling poetry install
https://github.com/ecmwf/eccodes-python/blob/2.45.0/setup.py#L20

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok amazing I'll try:)

Remove eccodeslib dependency from poetry and set DEPEND_ON_ECCODESLIB to 0.
@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

2 similar comments
@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 13, 2026

Hi @bcumming,
I am getting the same error again and again, but I don't see how my changes triggered them. Do you have an idea why the checks are failing?

Thanks and all the best.

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 17, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@nnellen
Copy link
Collaborator Author

nnellen commented Feb 20, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

Remove eccodeslib from poetry dependencies after installation.
@nnellen
Copy link
Collaborator Author

nnellen commented Feb 20, 2026

cscs-ci run alps;system=balfrin;uarch=zen3;uenv=fdb:5.18

@bcumming bcumming changed the title update eccodes fdb: update eccodes Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants