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

Add netCDF4 as a dependency and install netcdf4 #429

Closed
wants to merge 6 commits into from
Closed

Add netCDF4 as a dependency and install netcdf4 #429

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
970bdb4
List netCDF4 as a pygmt dependency
seisman May 17, 2020
98d292d
Also need to install netcdf4
seisman May 17, 2020
17ce11d
Merge branch 'master' into depend-netcdf4
seisman May 21, 2020
bfa9ff8
Move netcdf4 to optional dependency
seisman May 21, 2020
2e442ad
Merge branch 'master' into depend-netcdf4
seisman May 21, 2020
6df8b19
Add netcdf4 as a non-optional dependency
seisman May 21, 2020
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
3 changes: 2 additions & 1 deletion doc/install.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ PyGMT requires the following libraries:
* `numpy <http://www.numpy.org/>`__
* `pandas <https://pandas.pydata.org/>`__
* `xarray <http://xarray.pydata.org/>`__
* `netCDF4 <https://github.com/Unidata/netcdf4-python>`__
* `packaging <https://pypi.org/project/packaging/>`__

The following are optional (but recommended) dependencies:
Expand All @@ -79,7 +80,7 @@ First, we must configure conda to get packages from the
Now we can create a new conda environment with Python and all our dependencies installed
(we'll call it ``pygmt`` but you can change it to whatever you want)::

conda create --name pygmt python=3.6 pip numpy pandas xarray packaging gmt=6.0.0
conda create --name pygmt python=3.6 pip numpy pandas xarray netcdf4 packaging gmt=6.0.0
Copy link
Member

Choose a reason for hiding this comment

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

Once #430 is merged, would it be better to just instruct users to conda install pygmt since it'll pull in all the side dependencies as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, perhaps we should merge conda-forge/pygmt-feedstock#2 first, then instruct users to install pygmt with:

conda create --name pygmt pygmt


Activate the environment by running::

Expand Down