-
Notifications
You must be signed in to change notification settings - Fork 7
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
Conda dependency baseos for ctsm-fates in NorESM Galaxy project #30
Comments
Attempted to adopt the galaxy workflow of automating the container build. Currently unable to get the local tests to run. Following this documentation: https://docs.galaxyproject.org/en/latest/admin/special_topics/mulled_containers.html#building-docker-containers-for-local-conda-packages. Here is the error:
Got help from @bgruening about error on gitter, but I had to drop this to pivot to other work before I was able to resolve this. Test repo for this is here: https://github.com/glemieux/staged-recipes/tree/ctsm-fates. Based on recommendations from @annefou, attempted to submit a PR to conda-forge, but has failed checks for linux. Created an issue for that here: glemieux/staged-recipes#1 |
Looking at alternative method of providing existing dockerfile recipe to fit within the Galaxy workflow if possible. |
Yes I have seen this kind of error; right now I can't remember exactly why and when but i will look at it. |
@glemieux your conda package failed because it does not work under python3 can this be? |
The python errors in statistical_ensemble_test should not be a problem (I mean not for the conda build). The license is missing: |
in meta.yaml:
While now you have
And this is my fault because I made this error and corrected it later for fates-emerald. The conda-forge recipe for fates-emerald is at https://github.com/conda-forge/fates-emerald-feedstock |
But this is also hinting at a python2 problem: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=230158&view=logs&j=10bda42a-d420-5c28-a43f-bc4e66405e5c&t=bef775d0-a1a2-55a3-9721-17147029baf0&l=885 |
Yes, I also have it in my conda recipe... It passes even with these errors (this part is not used; additional tools still in python 2; I mean for print statements... Could easily be migrated to python 3; i am not sure who would be using these). |
Sorry for the late replies. Thanks for all the feedback. I'm guessing that this is low-hanging fruit that the CIME team just hasn't prioritized due to more pressing efforts. My guess is that they have to support python 2 by default on their supported machines anyway so their isn't a huge impetus to fix this. I'll double check with them though. Thanks for the pointers to the errors btw; I simply missed seeing that I could click through to the azure job report. |
After talking with one of the CTSM engineers at NCAR, he submitted an issue to address the python2 print statements: ESMCI/cime#3776 |
The Nordic Earth System Model (NorESM) is integrating FATES and working on utilizing its model within the Galaxy project to enable web-enabled, cloud-hosted research: NordicESMhub/galaxy-tools#39. Since the Galaxy project makes use of containers to promote reproducible science and cloud computing, we are collaborating with NorESM to bring containerized hlm-fates to Galaxy. The hope is that this will help further experience with and adoption of the FATES model.
Galaxy utilizes conda's package management infrastructure to distribute and maintain containers, specifically bioconda containers. As such, it is ideal that the dockerfile recipes utilize conda-based libraries for the containerized application dependencies.
The text was updated successfully, but these errors were encountered: