Skip to content

Conversation

@samsrabin
Copy link
Member

@samsrabin samsrabin commented Mar 4, 2025

Description of changes

This PR changes conda_env_ctsm_py.txt to install Python 3.13.2 instead of Python 3.7.9.

Notes to testers

The new ctsm_pylib is only expected to be compatible with Python tools in this branch (until it's merged). Even run_sys_tests will not work.

Handling existing ctsm_pylib

Rename your current ctsm_pylib and install the new one with that name:

./py_env_create -r ctsm_pylib_3_7_9

Install the new ctsm_pylib with a custom name:

./py_env_create -n ctsm_pylib_3_13_2

Specific notes

Contributors other than yourself, if any: None

CTSM Issues Fixed::

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? Resolves a pre-existing need for documentation (see #2771). ctsm-docs PR at ESCOMP/ctsm-docs#10.

Testing performed, if any:
As of 34ba0d6:

  • clm_pymods test suite:
  • Python unit tests
  • Python system tests
  • Black check
  • Pylint check

@samsrabin samsrabin added enhancement new capability or improved behavior of existing capability code health improving internal code structure to make easier to maintain (sustainability) labels Mar 4, 2025
@samsrabin samsrabin added this to the ctsm6.0.0 (code freeze) milestone Mar 4, 2025
@samsrabin samsrabin self-assigned this Mar 4, 2025
@samsrabin samsrabin force-pushed the ctsm_pylib_3.13.2 branch from 7ba8e40 to b1be486 Compare March 4, 2025 22:57
@samsrabin samsrabin force-pushed the ctsm_pylib_3.13.2 branch from 7f6f502 to 3048936 Compare March 5, 2025 04:31
@adrifoster
Copy link
Collaborator

This worked great for me!

@samsrabin
Copy link
Member Author

samsrabin commented Mar 27, 2025

  • Add pytest and coverage

@samsrabin samsrabin moved this from Ready to start (or start again) to In progress - master/b4b-dev in CTSM: Upcoming tags Apr 2, 2025
@github-project-automation github-project-automation bot moved this to Ready to start (or start again) in CTSM: Upcoming tags Apr 2, 2025
@samsrabin samsrabin added the bfb bit-for-bit label Apr 2, 2025
@samsrabin samsrabin requested a review from ekluzek April 7, 2025 22:06
@wwieder
Copy link
Contributor

wwieder commented Apr 9, 2025

@ekluzek I know you're at the workshop this week, but can you review / approve this PR, as it's holding up other work Sam's trying to do.

Copy link
Collaborator

@ekluzek ekluzek left a comment

Choose a reason for hiding this comment

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

We agree on the last conversation, so marking approve so you can do that on your own. I have something I ask about that you could choose to do or not, as you see fit.

@samsrabin samsrabin changed the title Update ctsm_pylib to 3.13.2 ctsm5.3.040: Update ctsm_pylib to 3.13.2 Apr 14, 2025
@samsrabin samsrabin merged commit c2a8c6c into ESCOMP:master Apr 15, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from In progress - master/b4b-dev to Done (non release/external) in CTSM: Upcoming tags Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bfb bit-for-bit blocker another issue/PR depends on this one code health improving internal code structure to make easier to maintain (sustainability) devops Development Operations to improve development throughput, E.g., adding GitHub Workflows enhancement new capability or improved behavior of existing capability PR status: needs testing

Projects

Status: Done (non release/external)

Development

Successfully merging this pull request may close these issues.

Add User's Guide section about ctsm_pylib Move ctsm_pylib to something beyond 3.7

6 participants