Skip to content

New NRL-only package py-lis-ncoda-utils - #2080

Open
climbfuji wants to merge 2 commits into
JCSDA:developfrom
climbfuji:feature/lis_ncoda
Open

New NRL-only package py-lis-ncoda-utils#2080
climbfuji wants to merge 2 commits into
JCSDA:developfrom
climbfuji:feature/lis_ncoda

Conversation

@climbfuji

@climbfuji climbfuji commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR adds a new package py-lis-ncoda-utils, which is available at NRL internally only. A new variant is added to the neptune-python-env virtual package and the neptune-dev template. Like for other NRL-internal packages, these variants are turned off in GitHub actions testing in this repo.

Dependencies

  • NRL-internal PR, after which tag 2.0.0 will be created for the new package py-lis-ncoda-utils

Issues addressed

NRL-internal issue, not visible here.

Applications affected

NEPTUNE

Systems affected

None

Testing

  • CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
    • GitHub actions CI tests pass
    • GitHub actions CI tests do not pass (provide explanation)
    • GitHub actions CI tests skipped (provide explanation if necessary)
  • New tests added: List and describe any new tests added to GitHub actions
    • ...
  • Additional testing: Add information on any additional tests conducted
    • Built py-lis-ncoda-utils with spack-stack and tested the utilities on real data.

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.
  • All necessary updates to the documentation (spack-stack wiki) will be made when this PR is merged

@climbfuji climbfuji self-assigned this Aug 7, 2026
Comment thread repos/spack_stack/spack_repo/spack_stack/packages/py_lis_ncoda_utils/package.py Outdated
# Restrict py-pandas to older versions compatible with py-numpy and py-xarray
py-pandas:
require:
- '@:2'

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This has no effect on the existing environments, because one or more existing packages provide the necessary constraint on py-pandas. But it surfaced as a necessary requirement when building just py-lis-ncoda-utils.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

But @mathomp4 may have to remove/relax that constraint in his PR, which updates py-numpy and py-xarray (these two packages, currently held back to versions several years old, require an older py-pandas).

@climbfuji
climbfuji marked this pull request as ready for review August 7, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants