Skip to content

Store Jinja environment in TemplateRenderer#88

Merged
nstauff merged 6 commits intowatts-dev:developmentfrom
paulromano:jinja-environment
Feb 10, 2023
Merged

Store Jinja environment in TemplateRenderer#88
nstauff merged 6 commits intowatts-dev:developmentfrom
paulromano:jinja-environment

Conversation

@paulromano
Copy link
Copy Markdown
Contributor

@paulromano paulromano commented Feb 9, 2023

Description

This PR includes a subtle change to our TemplateRenderer class, which now stores a jinja2.Environment instance. This may be needed down the line for some envisioned features (e.g., special Jinja filters to be used for MCNP) and otherwise doesn't affect functionality.

Two other small changes:

  • I added an input_file property to the PluginMCNP class
  • There was a small fix needed in plugin_dakota.py that was raising a warning during tests

Checklist:

  • My code follows the style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have updated the CHANGELOG.md file (if applicable)
  • I have successfully run examples that may be affected by my changes

Loading
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.

2 participants