Skip to content

Support Alternate Path for ngen Logs - #11

Merged
mxkpp merged 1 commit into
developmentfrom
maxkipp-log-paths
Mar 24, 2026
Merged

Support Alternate Path for ngen Logs#11
mxkpp merged 1 commit into
developmentfrom
maxkipp-log-paths

Conversation

@mxkpp

@mxkpp mxkpp commented Mar 19, 2026

Copy link
Copy Markdown

This adds support for an alternate path for ngen logs by persisting OS env var NGEN_RESULTS_DIR if already set.


This was tested in conjunction with the following PRs of other repositories:

For essential functionality changes:

NGWPC/nwm-cal-mgr#34

https://github.com/NGWPC/nwm-rte/pull/44

For pending EWTS integration:

NGWPC/ngen-forcing#121

And this branch of ngen which also includes updates to various extern submodules: https://github.com/NGWPC/ngen/tree/cmaynard_ewts_separate_repo

@cmaynard-ngwpc

Copy link
Copy Markdown

This is an elegant solution because the forecast manager simply sets the environment variable for the ewts ngen logging use and do not actually use it. They maintain their own internal reference for accessing the results directory. For them, it is simply a set it and forget it scenario.

@kyle-larkin kyle-larkin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Achieves the objectives - facilitating non-ngenCERF loglevel setting, and allowing logfiles to be written to alternate paths.

@mxkpp
mxkpp merged commit ebd34b5 into development Mar 24, 2026
7 checks passed
@mxkpp
mxkpp deleted the maxkipp-log-paths branch March 24, 2026 13:29
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.

3 participants