You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It appears that #4618 brought in the import of a non-standard python module (evv4esm) into cime. Up until this point we have been very careful not to allow import of modules that are not in the standard python distribution and this restriction has served us very well. This import is causing a number of tests in the scripts regression tests to fail for cesm which does not import this module. It took me quite a while to understand why these tests were passing in the github test container but not on my local systems. I now understand that the container imports this module through the file test-requirements.txt. But this file is not invoked in local testing.
Perhaps if there had been some collaboration or communication regarding this module with the cesm team we would be using it too but since we are not I kindly request that this be removed as a testing requirement for cesm.
The text was updated successfully, but these errors were encountered:
That was a mistake and the module should not have been made part of CIME. Tests that need extra modules are supposed to run in the CIMEenv conda which can include those.
It appears that #4618 brought in the import of a non-standard python module (evv4esm) into cime. Up until this point we have been very careful not to allow import of modules that are not in the standard python distribution and this restriction has served us very well. This import is causing a number of tests in the scripts regression tests to fail for cesm which does not import this module. It took me quite a while to understand why these tests were passing in the github test container but not on my local systems. I now understand that the container imports this module through the file test-requirements.txt. But this file is not invoked in local testing.
Perhaps if there had been some collaboration or communication regarding this module with the cesm team we would be using it too but since we are not I kindly request that this be removed as a testing requirement for cesm.
The text was updated successfully, but these errors were encountered: