Skip to content

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Mar 4, 2025

This clean-up PR also switches from find_executable() from distutil (which is deprecated) with shutil.which().

Checklist

  • Testing comment in the PR documents testing used to verify the changes

@xylar xylar added the clean up label Mar 4, 2025
@xylar xylar self-assigned this Mar 4, 2025
@xylar xylar force-pushed the fix-decode-timedelta-warning branch from 5074205 to 4555c40 Compare March 4, 2025 20:45
@xylar
Copy link
Collaborator Author

xylar commented Mar 4, 2025

Testing

I successfully ran the developer test suite:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xylar/analysis_testing/chrysalis/fix-decode-timedelta-warning/
I grepped through the job output files and log files, and the FutureWarning entries are all gone!

@xylar xylar merged commit e33a335 into MPAS-Dev:develop Mar 4, 2025
5 checks passed
@xylar xylar deleted the fix-decode-timedelta-warning branch March 4, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant