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

Jinja error during experiment creation on WCOSS #3087

Closed
WalterKolczynski-NOAA opened this issue Nov 12, 2024 · 5 comments · Fixed by #3053
Closed

Jinja error during experiment creation on WCOSS #3087

WalterKolczynski-NOAA opened this issue Nov 12, 2024 · 5 comments · Fixed by #3053
Labels
bug Something isn't working

Comments

@WalterKolczynski-NOAA
Copy link
Contributor

What is wrong?

Following the early November upgrade to Cactus, experiment creation now fails due to a non-existent jinja attribute:

The create_experiment command (./create_experiment.py -y ../ci/cases/pr/C48_ATM.yaml --overwrite) failed with a non-zero status.  Output:
Traceback (most recent call last):
  File "./create_experiment.py", line 33, in <module>
    from wxflow import AttrDict, parse_j2yaml, Logger, logit
  File "/lfs/h2/emc/ptmp/walter.kolczynski/PR/PR_3053/global-workflow/workflow/wxflow/__init__.py", line 13, in <module>
    from .jinja import Jinja
  File "/lfs/h2/emc/ptmp/walter.kolczynski/PR/PR_3053/global-workflow/workflow/wxflow/jinja.py", line 16, in <module>
    @jinja2.pass_eval_context
AttributeError: module 'jinja2' has no attribute 'pass_eval_context'

What should have happened?

setup_expt.py should create the experiment direction correctly.

What machines are impacted?

WCOSS2

What global-workflow hash are you using?

3FB68397

Steps to reproduce

  1. Clone on Cactus
  2. Load the gw_setup module
  3. Attempt to create an experiment

Additional information

Presumably this will temporarily go away when they switch production, the reappear after Dogwood is upgraded the week of Nov 18.

See also #3086

Do you have a proposed solution?

No response

@WalterKolczynski-NOAA WalterKolczynski-NOAA added the bug Something isn't working label Nov 12, 2024
@DavidHuber-NOAA
Copy link
Contributor

Is there an issue open with the help desk?

@aerorahul
Copy link
Contributor

The fix for this is simple. Load the python module in module_gwsetup.wcoss2.lua. For some reason, wcoss2 was using system python and not sure why that worked.

@JongilHan66
Copy link

The fix for this is simple. Load the python module in module_gwsetup.wcoss2.lua. For some reason, wcoss2 was using system python and not sure why that worked.

@aerorahul Could you let me know how to load python module in module_gwsetup.wcoss2.lua specifically?

@aerorahul
Copy link
Contributor

The fix for this is simple. Load the python module in module_gwsetup.wcoss2.lua. For some reason, wcoss2 was using system python and not sure why that worked.

@aerorahul Could you let me know how to load python module in module_gwsetup.wcoss2.lua specifically?

@JongilHan66
A fix was merged in #3053. The change can be seen here.

@JongilHan66
Copy link

The fix for this is simple. Load the python module in module_gwsetup.wcoss2.lua. For some reason, wcoss2 was using system python and not sure why that worked.

@aerorahul Could you let me know how to load python module in module_gwsetup.wcoss2.lua specifically?

@JongilHan66 A fix was merged in #3053. The change can be seen here.

@aerorahul Thanks!!

WalterKolczynski-NOAA added a commit to WalterKolczynski-NOAA/global-workflow that referenced this issue Nov 22, 2024
)

This moves all machine-specific options to the workflow/hosts files from
the config.* files.

This also turns HPSS archiving on for WCOSS2 by default.
This turns OFF UFS DA tests on WCOSS2.

Resolves NOAA-EMC#2942
Resolves NOAA-EMC#3087

---------

Co-authored-by: Walter Kolczynski - NOAA <[email protected]>
Co-authored-by: Rahul Mahajan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants