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.

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

NetCDF preview #6

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

NetCDF preview #6

wants to merge 18 commits into from

Conversation

tcnichol
Copy link

Here is the netCDF preview. There is also a pull request in extractors-geo but wanted to put it here as well since it is a clowder2-demo.

I checked in a sample file to test that was smaller, larger files are shared here:

https://drive.google.com/drive/folders/1ouc-jq8L21uwQzHNvP2VRrePxec5lO9i

@tcnichol tcnichol linked an issue Oct 13, 2023 that may be closed by this pull request
@tcnichol tcnichol requested a review from a team October 13, 2023 18:47
Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

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

Im getting ModuleNotFoundError: No module named 'mpl_toolkits.basemap'
Does this library need to be included in requirement.txt?

preview.netcdf/extractor_info.json Outdated Show resolved Hide resolved
preview.netcdf/ncsa.geo.netcdf.extractor.py Show resolved Hide resolved
preview.netcdf/requirements.txt Show resolved Hide resolved
preview.netcdf/requirements.txt Outdated Show resolved Hide resolved
@tcnichol
Copy link
Author

Im getting ModuleNotFoundError: No module named 'mpl_toolkits.basemap' Does this library need to be included in requirement.txt?

I did not get that error locally, but I notice it showed up in the Docker image when I tried to run.

Since I was having dependency issues running things related to NetCDF in Docker I am switching to using a conda environment. Seeing if I can resolve this.

@ddey2 ddey2 self-requested a review October 26, 2023 17:40
Copy link
Member

@ddey2 ddey2 left a comment

Choose a reason for hiding this comment

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

Can you add netcdfutils, matplotlib packages? I am seeing a lot of imports like netCDF4, mpl_toolkits etc which seems like not being used.

@tcnichol
Copy link
Author

tcnichol commented Nov 6, 2023

As of now, I have been unable to get this to run in PyCharm using the Docker image as the run time environment. Looking into this.

In the meantime, it does work if you use the conda environment. I have also removed requirements.txt and replaced it with the environment.yml for the conda environment.

@tcnichol
Copy link
Author

tcnichol commented Feb 8, 2024

Can you add netcdfutils, matplotlib packages? I am seeing a lot of imports like netCDF4, mpl_toolkits etc which seems like not being used.

This should be changed in the environment.yml file now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

netCDF visualization
3 participants