Skip to content

Use EWTS repository Python package - #20

Merged
cmaynard-ngwpc merged 4 commits into
developmentfrom
cmaynard_NGWPC_10000_use_ewts_package
Apr 16, 2026
Merged

Use EWTS repository Python package#20
cmaynard-ngwpc merged 4 commits into
developmentfrom
cmaynard_NGWPC_10000_use_ewts_package

Conversation

@cmaynard-ngwpc

Copy link
Copy Markdown

Summary

Refactor to use the EWTS-based logging by centralizing logger initialization, standardizing log naming, and ensuring consistent usage across the codebase.

Additions

  • Centralized logger initialization utility in utils.py for consistent logger configuration and reuse.
  • Added prefixes for ngen logs by setting the new NGEN_LOG_FILE_PREFIX environment variable

Updates

  • Migrated remaining logging to use EWTS across the codebase.
  • Standardized log naming by prefixing ngen stdout/stderr logs with the job ID for better traceability
  • Refactored logger setup to rely on the new shared utility.
  • Added cache bust args to the Dockerfile

Removals

  • Removed redundant logger initialization in favor of the centralized utility.

@cmaynard-ngwpc
cmaynard-ngwpc merged commit 433f2ca into development Apr 16, 2026
7 checks passed
@cmaynard-ngwpc
cmaynard-ngwpc deleted the cmaynard_NGWPC_10000_use_ewts_package branch April 16, 2026 23:51
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