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.

Sign up for GitHub

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

Jump to bottom

Remove cdtime as a library dependency #615

Closed
tomvothecoder opened this issue Jul 18, 2022 · 1 comment
Closed

Remove cdtime as a library dependency #615

tomvothecoder opened this issue Jul 18, 2022 · 1 comment
Assignees
@tomvothecoder
Labels
cdat-migration-fy24 CDAT Migration FY24 Task
Milestone
FY23 Q3/Q4 (04/01...

Comments

@tomvothecoder
Copy link
Collaborator

cdtime is only directly imported in analysis_data_preprocess scripts.

There is already a dedicated step for creating an environment specifically for preprocessing:

e3sm_diags/analysis_data_preprocess/README

Line 3 in e15610c

conda create -n obs_env cdtime cdms2 cdutil numpy pytz nco cdo python=2.7 -c conda-forge -c cdat

It can be removed as a library dependency easily.
/conda-envs
@tomvothecoder tomvothecoder self-assigned this Jul 18, 2022
@tomvothecoder tomvothecoder changed the title Remove cdtime Remove cdtime as a library dependency Jul 18, 2022
@tomvothecoder tomvothecoder mentioned this issue Nov 10, 2022
Closed
3 tasks
@chengzhuzhang
Copy link
Contributor

closed by #627

@tomvothecoder tomvothecoder added cdat-migration-fy24 CDAT Migration FY24 Task and removed refactor labels Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomvothecoder tomvothecoder

Labels
cdat-migration-fy24 CDAT Migration FY24 Task
Projects
None yet
Milestone
FY23 Q3/Q4 (04/01/24-09/30/23)
Development

No branches or pull requests
2 participants
@chengzhuzhang @tomvothecoder