rigoudyg / climaf

CliMAF - a Climate Model Analysis Framework - doc at : http://climaf.readthedocs.org/
Other
16 stars 7 forks source link

V2.0.2 #211

Closed rigoudyg closed 2 years ago

rigoudyg commented 3 years ago

This pull request prepare the next version of CliMAF. All developments included in the next version will be put into.

rigoudyg commented 2 years ago

@senesis, I have make a few changes in order to define cache variables in env.environment module. It seems to work. Could you test on your side?

senesis commented 2 years ago

@senesis, I have make a few changes in order to define cache variables in env.environment module. It seems to work. Could you test on your side?

@rigoudyg, The test suite is working fine on Clclad. Do you expect more specific tests ?

rigoudyg commented 2 years ago

@senesis, I have make a few changes in order to define cache variables in env.environment module. It seems to work. Could you test on your side?

@rigoudyg, The test suite is working fine on Clclad. Do you expect more specific tests ?

@senesis Thank you for testing the changes. I do not manage to rebase your PR #215 on that branch, the module climaf.projects.optimize.py is not able to "view" the variables defined in the env.environment and I do not manage to modified it to resolve this issue.

senesis commented 2 years ago

I do not manage to rebase your PR #215 on that branch, the module climaf.projects.optimize.py is not able to "view" the variables defined in the env.environment and I do not manage to modified it to resolve this issue.

@rigoudyg : This is why I moved the cache dir value back to cache.py, after struggling for a while....

senesis commented 2 years ago

I do not manage to rebase your PR #215 on that branch, the module climaf.projects.optimize.py is not able to "view" the variables defined in the env.environment and I do not manage to modified it to resolve this issue.

@rigoudyg : This is why I moved the cache dir value back to cache.py, after struggling for a while....

I merged the material from PR #215 and current V2.0.2 in another branch, and issued PR #235, which replaces #215