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

Add gnuplot to the conda environment #483

Merged
merged 4 commits into from
Mar 1, 2024

Conversation

dabail10
Copy link
Contributor

For detailed information about submitting Pull Requests (PRs) to the CICE-Consortium,
please refer to: https://github.com/CICE-Consortium/About-Us/wiki/Resource-Index#information-for-developers

PR checklist

  • Short (1 sentence) summary of your PR:
    Just adds gnuplot to the conda environment.
  • Developer(s):
    dabail10 (D. Bailey)
  • Suggest PR reviewers from list in the column to the right.
  • Please copy the PR test results link or provide a summary of testing completed below.

Tested on my Mac conda environment.

[cgdm-hurricane:~/icepack-tutorial/icepack_master] dbailey% conda env create -f configuration/scripts/machines/environment.yml
Channels:

  • conda-forge
    Platform: osx-64
    Collecting package metadata (repodata.json): done
    Solving environment: done

Downloading and Extracting Packages:

Preparing transaction: done
Verifying transaction: done
Executing transaction: done

To activate this environment, use

$ conda activate icepack

To deactivate an active environment, use

$ conda deactivate

[cgdm-hurricane:/icepack-tutorial/icepack_master] dbailey% conda activate icepack
(icepack) [cgdm-hurricane:
/icepack-tutorial/icepack_master] dbailey% which gnuplot
/Users/dbailey/miniconda3/envs/icepack/bin/gnuplot

  • How much do the PR code changes differ from the unmodified code?
    • bit for bit
    • different at roundoff level
    • more substantial
  • Does this PR create or have dependencies on CICE or any other models?
    • Yes
    • No
  • Does this PR add any new test cases?
    • Yes
    • No
  • Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/.)
    • Yes
    • No, does the documentation need to be updated at a later time?
      • Yes
      • No
  • Please provide any additional information or relevant details below:
    This fix allows the timeseries.csh script to be used for Icepack standalone runs.

Copy link
Member

@phil-blain phil-blain left a comment

Choose a reason for hiding this comment

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

Thanks Dave!

@dabail10
Copy link
Contributor Author

I'm not sure why this is failing. It seems to be building the conda environment correctly ...

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

Could you comment out the gnuplot addition then push that change. Lets see if that fails ghactions. If not, turn it back on then resubmit, lets see if the failure is a problem in ghactions or gnuplot. It seems to be failing on debug+histcdf. I wonder if adding gnuplot is messing with the netcdf somehow.

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

Did you build and run the icepack test suite on your mac or just verify that gnuplot was part of the environment?

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

Looks like ghactions may have changed and now things are not working, see #484. Occurs far too often. Let me look into it.

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

@dabail10, if you get a chance, can you run the io_suite on your mac laptop and let me know if you get any failures. I'm having trouble duplicating this error with my mac env (older OS), and I'm not getting much useful info from the log files on github actions. I'm updating my conda env now and will try a laptop shortly to see if I can make any progress. Just wondering if a newer mac OS which you might have will pick this up.

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

I updated my conda env on my mac desktop, I'm seeing the failure now. Will try to debug. @dabail10, don't need anything from you at the moment.

@dabail10
Copy link
Contributor Author

This worked on my mac:

#-------
#repo = origin http://github.com/ESCOMP/Icepack (push)
#bran = On branch main
#hash = f6ff8f7
#hshs = f6ff8f7
#hshu = Tony Craig [email protected]
#hshd = Fri Nov 17 09:22:49 2023 -0800
#suit = io_suite
#date = 2024-02-29
#time = 21:05:13
#mach = conda
#user = dbailey
#vers = ICEPACK_1.4.0
#-------
#---
PASS conda_macos_restart_col_1x1_debug_ionetcdf build
PASS conda_macos_restart_col_1x1_debug_ionetcdf initialrun
PASS conda_macos_restart_col_1x1_debug_ionetcdf run
PASS conda_macos_restart_col_1x1_debug_ionetcdf test
#---
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_run1year build
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_run1year run
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_run1year test
#---
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf build
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf initialrun
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf run
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf test
#---
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_run1year build
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_run1year run
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_run1year test
#---
PASS conda_macos_restart_col_1x1_debug_ionetcdf_restcdf build
PASS conda_macos_restart_col_1x1_debug_ionetcdf_restcdf initialrun
PASS conda_macos_restart_col_1x1_debug_ionetcdf_restcdf run
PASS conda_macos_restart_col_1x1_debug_ionetcdf_restcdf test
#---
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_restcdf_run1year build
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_restcdf_run1year run
PASS conda_macos_smoke_col_1x1_diag1_ionetcdf_restcdf_run1year test
#---
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf_restcdf build
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf_restcdf initialrun
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf_restcdf run
PASS conda_macos_restart_col_1x1_debug_histcdf_ionetcdf_restcdf test
#---
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_restcdf_run1year build
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_restcdf_run1year run
PASS conda_macos_smoke_col_1x1_diag1_histcdf_ionetcdf_restcdf_run1year test

Descriptors:
PASS - successful completion
COPY - previously compiled code was copied for new test
MISS - comparison data is missing
PEND - status is undetermined; test may still be queued, running, or timed out
FAIL - test failed

28 measured results of 28 total results
28 of 28 tests PASSED
0 of 28 tests PENDING
0 of 28 tests MISSING data
0 of 28 tests FAILED

@apcraig
Copy link
Contributor

apcraig commented Feb 29, 2024

OK, it did for me too until I updated to the latest conda env.

You can update your conda env by doing something like

conda env create -f configuration/scripts/machines/environment.yml --force

from the top of cice or icepack.

I've decided this is a compiler issue. It's failing in nf90_create when debug is on, in particular with ffpe-trap=invalid. So, I'm going to turn off that debug flag. Will update #484 with this fix, we can merge that, then we can restart ghactions testing on this PR.

@apcraig
Copy link
Contributor

apcraig commented Mar 1, 2024

@dabail10, can you update your branch from main then push the update. That should allow ghactions to run.

@dabail10
Copy link
Contributor Author

dabail10 commented Mar 1, 2024

I changed a comment line and committed and pushed, but it did not restart GHActions.

@dabail10
Copy link
Contributor Author

dabail10 commented Mar 1, 2024

I just commented out the gnuplot line and comitted / pushed and still not working.

@apcraig
Copy link
Contributor

apcraig commented Mar 1, 2024

ghactions ran and passed. Can you update the branch again with gnuplot.

@apcraig apcraig merged commit 7e529b1 into CICE-Consortium:main Mar 1, 2024
2 checks passed
dabail10 added a commit to dabail10/Icepack that referenced this pull request Mar 14, 2024
* Add gnuplot to the conda environment

This fix allows the timeseries.csh script to be used for Icepack standalone runs.
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.

3 participants