PCMDI / pcmdi_metrics

Open-source Python package for Systematic Evaluation of Climate and Earth System Models
http://pcmdi.github.io/pcmdi_metrics/
BSD 3-Clause "New" or "Revised" License
103 stars 40 forks source link

[Bug]: io.base download_sample_data_files not updated in conda version 3.5 #1114

Closed ShihengDuan closed 2 months ago

ShihengDuan commented 2 months ago

What happened?

I'm following the Demo0 but it shows pcmdi_metrics.io.base doesn't have module download_sample_data_files. I have to git clone the repo and pip install to solve this problem.

Then at from download_sample_data import generate_parameter_files I have a similar problem that No module named 'download_sample_data'. It works when I put my notebook under the Demo path. I believe this is not related to the PMP version.

What did you expect to happen? Are there are possible answers you came across?

No response

Minimal Complete Verifiable Example (MVCE)

No response

Relevant log output

No response

Anything else we need to know?

No response

Environment

When I use conda install, it shows version 3.5 when I check with conda list pcmdi_metrics. But it shows 3.4.1 with pip show pcmdi_metrics, and in the notebook it shows 3.3.1. After I used pip install ., it shows the correct version with pip show pcmdi_metrics.

(pcmdi) duan0000@perlmutter:login16:/pscratch/sd/d/duan0000/neuralgcm/results> pip show pcmdi_metrics
Name: pcmdi_metrics
Version: 3.4.1
Summary: model metrics tools
Home-page: http://github.com/PCMDI/pcmdi_metrics
Author: PCMDI
Author-email: 
License: 
Location: /global/u2/d/duan0000/.conda/envs/pcmdi/lib/python3.10/site-packages
Requires: 
Required-by: 

(pcmdi) duan0000@perlmutter:login16:/pscratch/sd/d/duan0000/neuralgcm/results> pip show pcmdi_metrics
Name: pcmdi_metrics
Version: 3.5
Summary: model metrics tools
Home-page: http://github.com/PCMDI/pcmdi_metrics
Author: PCMDI
Author-email: 
License: 
Location: /global/u2/d/duan0000/.conda/envs/pcmdi/lib/python3.10/site-packages
Requires: 
Required-by: 
lee1043 commented 2 months ago

@ShihengDuan thank you for the report! @acordonez and I will take this a look.

lee1043 commented 2 months ago

@ShihengDuan I couldn't reproduce this error. Could you retry after reinstalling the PMP and if the issue is still persistence?

acordonez commented 2 months ago

Discussed offline, issue resolved on user's end.