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

Issue #2016 update the Python versions used by the METplus analysis t… #2044

Merged
merged 3 commits into from
Feb 16, 2023
Merged

Issue #2016 update the Python versions used by the METplus analysis t… #2044

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
cc36017
Issue #2016 update the Python versions used by the METplus analysis t…
bikegeek Feb 9, 2023
e546d92
Issue #2016 update netcdf version to be consistent with NCO version
bikegeek Feb 10, 2023
909ee8d
Issue #2016 Remove yaml
bikegeek Feb 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 16 additions & 16 deletions docs/Users_Guide/overview.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,7 @@ METplus Components Python Requirements
(h5py)
<../generated/met_tool_wrapper/PCPCombine/PCPCombine_python_embedding.html>`_
* - imutils
- >=0.5.3
- >=0.5.4
- METplotpy :raw-html:`<br />`
- https://pypi.org/project/imutils/
- A series of convenience :raw-html:`<br />`
Expand All @@ -223,7 +223,7 @@ METplus Components Python Requirements
and much more easier
-
* - imageio
-
- >=2.19.3
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 2.19.3 in plotpy requirements.txt, but 2.6.1 in calcpy requirements.txt. Should we update the calcpy requirements.txt file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Created issue in METcalcpy to update the versions in the requirements.txt:
dtcenter/METcalcpy#276

- METcalcpy, :raw-html:`<br />`
METplotpy
- https://pypi.org/project/imageio/
Expand All @@ -236,7 +236,7 @@ METplus Components Python Requirements
and scientific formats
-
* - lxml
-
- >=4.9.1
- METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
METdataio
Expand All @@ -246,7 +246,7 @@ METplus Components Python Requirements
libxml2 and libxslt
-
* - matplotlib
- >=3.3.4
- >=3.5.2
- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy
Expand Down Expand Up @@ -349,7 +349,7 @@ METplus Components Python Requirements
(metplotpy)
<../generated/model_applications/s2s/TCGen_fcstGFSO_obsBDECKS_GDF_TDF.html>`_
* - metpy
-
- >=1.3.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 1.3.1 in calcpy and plotpy nco_requirements.txt, but as 1.1.0 in calcpy requirements.txt. Should we update the calcpy requirements.txt file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dtcenter/METcalcpy#276

- METplus wrappers
- https://www.unidata.ucar.edu/software/metpy/
- a collection of tools :raw-html:`<br />`
Expand All @@ -371,7 +371,7 @@ METplus Components Python Requirements
\**REQUIRES Python 3.7
-
* - netCDF4
- >=1.5.4
- >=1.6.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 1.6.0 in METplotpy nco_requirements.txt and in plotpy requirements.txt, but 1.5.7 in calcpy requirements.txt and 1.6.2 in calcpy nco_requirements.txt. Should we update the calcpy requirements.txt and calcpy nco_requirements.txt or should the value here be 1.6.2 and should we update everywhere else?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dtcenter/METcalcpy#276

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's prudent to update to the latest version, so let's keep 1.6.2 and update in METplotpy and METcalcpy. I will put a note in METplotpy dtcenter/METplotpy#310 (update to Python 3.10 to use netCDF 1.6.2)

Copy link
Collaborator

Choose a reason for hiding this comment

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

@bikegeek I defer to you on this one. If that is what you think is best, that works for me. This is PR approved. :)

- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy
Expand All @@ -380,7 +380,7 @@ METplus Components Python Requirements
the netCDF C library
- For using MET Python embedding functionality in use cases
* - numpy
- >=1.19.2
- >=1.23.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 1.22.0 in nco_requirements.txt, 1.22.3 in calcpy requirements.txt. I'm not sure which one we need and if we should update elsewhere.

- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
Expand All @@ -394,7 +394,7 @@ METplus Components Python Requirements
Fourier transforms, and more.
- For using MET Python embedding functionality in use cases
* - pandas
- >=1.0.5, <=1.2.3 (METdataio)
- >=1.5.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

1.5.1 in calcpy, plotpy and dataio nco_requirements.txt and 1.5.1 in plotpy and dataio requirements.txt, but 1.2.3 in calcpy requirements.txt. Should we update in calcpy requirements.txt?

- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
Expand All @@ -409,15 +409,15 @@ METplus Components Python Requirements
language
- For using MET Python embedding functionality in use cases
* - pint
- >=0.18
- >=0.19.2
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 0.19.2 in calcpy and plotpy nco_requirements.txt and in plotpy requirements.txt, but 0.18 in calcpy requirements.txt. Should we update calcpy requirements.txt?

- METcalcpy
- https://github.com/hgrecco/pint
- Python package to define, :raw-html:`<br />`
operate and manipulate :raw-html:`<br />`
physical quantities
-
* - plotly
- >=4.9.0
- >=5.9.0
- METcalcpy, :raw-html:`<br />`
METplotpy
- https://github.com/plotly/plotly.py
Expand Down Expand Up @@ -458,7 +458,7 @@ METplus Components Python Requirements
(pylab)
<../generated/model_applications/s2s/UserScript_fcstGFS_obsERA_WeatherRegime.html>`_
* - pymysql
-
- >=1.0.2
- METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
METdataio
Expand Down Expand Up @@ -506,7 +506,7 @@ METplus Components Python Requirements
(pyresample)
<../generated/model_applications/marine_and_cryosphere/GridStat_fcstRTOFS_obsSMAP_climWOA_sss.html>`_
* - pytest
- >=5.2.1
- >=7.2.0
- METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
METdataio
Expand All @@ -528,7 +528,7 @@ METplus Components Python Requirements
libraries like Plotly
-
* - pyyaml
- >=5.3.1
- >=6.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 6.0 in calcpy, plotpy, and dataio nco_requirements.txt and 6.0 in plotpy and dataio requirements.txt, but 5.4.1 in calcpy requirements.txt. Should we update calcpy requirements.txt?

- METcalcpy, :raw-html:`<br />`
METplotpy, :raw-html:`<br />`
METdataio
Expand All @@ -538,7 +538,7 @@ METplus Components Python Requirements
programming language
-
* - scikit-image
- >=0.16.2
- >=0.19.3
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 0.18.1 in calcpy requirements.txt. Should we updated calcpy requirements.txt?

- METcalcpy, :raw-html:`<br />`
METplotpy
- https://scikit-image.org
Expand Down Expand Up @@ -572,7 +572,7 @@ METplus Components Python Requirements
(scikit-learn)
<../generated/model_applications/marine_and_cryosphere/GridStat_fcstRTOFS_obsSMAP_climWOA_sss.html>`_
* - scipy
- >=1.5.1
- >=1.8.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is listed as 1.8.1 in calcpy and plotpy nco_requirements.txt and 1.8.1 in plotpy requirements, but 1.8.0 in calcpy requirements.txt. Should we update calcpy requirements

- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy
Expand Down Expand Up @@ -635,7 +635,7 @@ METplus Components Python Requirements
(sklearn)
<../generated/model_applications/s2s/UserScript_fcstGFS_obsERA_WeatherRegime.html>`_
* - xarray
- >=0.17.0
- >=2022.3.0
- METplus wrappers, :raw-html:`<br />`
METcalcpy, :raw-html:`<br />`
METplotpy
Expand Down