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

Configuration for [email protected] and [email protected] #1240

Merged
merged 16 commits into from
Aug 22, 2024

Conversation

srherbener
Copy link
Collaborator

@srherbener srherbener commented Aug 15, 2024

Summary

This PR contains an initial configuration for the new bufr_query library from NOAA-EMC

Testing

  • AWS Ubuntu 20.24 EC2 instance gcc (@srherbener)
  • Discover SCU16 intel (@srherbener)
    • spack-stack built successfully
    • Discover is down today for Milan upgrades. Since jedi-bundle built and tested okay on teh AWS instance, I'm calling it good for this test.

Applications affected

JEDI, There are likely some NOAA applications affected, but I'm not clear on which ones.

Systems affected

HPCs, MacOS

Dependencies

If there are PRs that need to be merged before or along with this one, please add "- [ ] waiting on LINK_TO_PR" for each of them

Issue(s) addressed

Resolves #1182

Checklist

  • This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.

@srherbener
Copy link
Collaborator Author

My testing on the AWS instance ran into this issue: jcsda-internal/jedi-bundle/issues/95, but I also found that the bufr-query package is installing its python extension in the lib/pythonX.Y directory instead of the expected lib/pythonX.Y/site-packages directory.

What's the proper way to fix this?

  • Adjust PYTHONPATH in the module file for bufr-query
  • Change the bufr-query CMake configuration to install in the site-packages directory

Thanks!

@climbfuji
Copy link
Collaborator

My testing on the AWS instance ran into this issue: JCSDA-internal/jedi-bundle/issues/95, but I also found that the bufr-query package is installing its python extension in the lib/pythonX.Y directory instead of the expected lib/pythonX.Y/site-packages directory.

What's the proper way to fix this?

  • Adjust PYTHONPATH in the module file for bufr-query
  • Change the bufr-query CMake configuration to install in the site-packages directory

Thanks!

Kind of surprising, the Python extension should be handled correctly by spack (i.e. the spack modulefile should contain the correct pythonpath). Maybe it is a cmake configuration issue?

@climbfuji
Copy link
Collaborator

@srherbener I merged the spack PR. Feel free to push the submodule pointer update and .gitmodules reversal, but please add [skip ci] as an empty last commit so that we don't trigger CI runs. We've got a bit of a backlog of PRs to merge and CI takes a lot of time at the moment. Thanks!

@srherbener
Copy link
Collaborator Author

Sorry, I tried to do the "skip ci" but it didn't seem to take effect.

@climbfuji
Copy link
Collaborator

Sorry, I tried to do the "skip ci" but it didn't seem to take effect.

[skip ci] is the correct syntax

@srherbener srherbener changed the title Initial configuration for the new bufr_query library Configuration for [email protected] and [email protected] Aug 21, 2024
@srherbener srherbener mentioned this pull request Aug 21, 2024
3 tasks
@climbfuji
Copy link
Collaborator

@srherbener I resolved the merge conflicts in jedi-ci.yaml, please check. I want to confirm with you that bufr_query is not supposed to be in jedi-ci.yaml (currently it isn't).

@srherbener
Copy link
Collaborator Author

@climbfuji thanks for the note about the missing bufr-query entry in the jedi-ci config. I do want that in. I noticed that this PR is out of date with develop, so I will add the bufr-query entry and merge develop.

@srherbener srherbener force-pushed the feature/intro-bufr-query branch from f6dc438 to 2b9c11c Compare August 22, 2024 16:11
@srherbener
Copy link
Collaborator Author

This is ready for review now. Thanks!

@srherbener srherbener requested a review from eap August 22, 2024 18:07
@srherbener srherbener added the INFRA JEDI Infrastructure label Aug 22, 2024
@climbfuji climbfuji enabled auto-merge (squash) August 22, 2024 19:22
@climbfuji climbfuji disabled auto-merge August 22, 2024 19:36
@climbfuji climbfuji merged commit cee10bb into JCSDA:develop Aug 22, 2024
8 checks passed
@srherbener srherbener deleted the feature/intro-bufr-query branch August 22, 2024 19:37
@srherbener
Copy link
Collaborator Author

@climbfuji thanks for all of your help with the eckit and bufr-query work. It's much appreciated!

DavidHuber-NOAA added a commit to DavidHuber-NOAA/spack-stack that referenced this pull request Aug 26, 2024
* jcsda/develop:
  Bump Python to 3.11.7 (JCSDA#1217)
  Add a clause in the cleanup to fix directory permissions (JCSDA#1273)
  Bug fix: configure neptune-env variants in three templates: neptune-dev, skylab-dev, unified-dev (JCSDA#1268)
  Update site configs for Atlantis, Narwhal, Nautilus (JCSDA#1266)
  Configuration for [email protected] and [email protected] (JCSDA#1240)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INFRA JEDI Infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[INSTALL]: New bufr_query library from EMC
2 participants