Skip to content

plots on Lambert conformal projection#204

Draft
weihuang-jedi wants to merge 8 commits into
mainfrom
feature/plots-lombert-conformal-projection
Draft

plots on Lambert conformal projection#204
weihuang-jedi wants to merge 8 commits into
mainfrom
feature/plots-lombert-conformal-projection

Conversation

@weihuang-jedi

Copy link
Copy Markdown

Description:

Add a python script to plot EAGEL LAM on Lambert conformal projection, a yaml template, and a run script.
Issue: #203

Type of change:

  • Bug fix
  • [x ] Enhancement / new feature
  • Refactor / cleanup
  • Documentation
  • CI/CD or tooling
  • Subcomponent update
  • Other:

Area(s) affected

  • nested_eagle workflow
  • Training
  • Inference
  • Verification / evaluation (via WXVX)
  • Data prep / UFS2ARCO
  • Config (YAML)
  • [x ] Plotting / post-processing
  • Infrastructure / Slurm scripts
  • Other:

Commit Requirements:

  • This PR addresses a relevant NOAA-EPIC/EAGLE issue (if not, create an issue); a person responsible for submitting the update has been assigned to the issue (link issue)
  • Fill out all sections of this template.
  • [x ] I have performed a self-review of my own code
  • [ x] My changes generate no new warnings
  • I have made corresponding changes to the system documentation if necessary
  • I checked whether this PR requires subcomponent PRs and completed the subcomponent checklist below

Testing / Verification:

  • [ x] I ran and/or verified the changes (or provided a test plan)

edit file "config.yaml.template", and gen-vis.sh for your own date/env, and then run script "gen-vis.sh".

Runtime Environment:

  • System/HPC: ursa
  • Account/role: epic
  • Conda env: EAGLE/visualization (mamba activate visualization).
  • Key versions (optional):
    • python --version: Python 3.13.0
    • wxvx --version (if applicable):
    • MET version (if applicable):

Commit Message:

"add vis scripts"

Subcomponent Pull Requests:

  • [ x] No subcomponent PRs are required for this change.
  • Required subcomponent PRs are linked below.
  • EAGLE: NOAA-EPIC/EAGLE#
  • UFS2ARCO: NOAA-PSL/UFS2ARCO#
  • WXVX (verification): NOAA-EPIC/WXVX#

NOTE:

This PR just want to show the new script which generate EAGLE LAM on Lambert Conformal projection maps, and see if people like it.

@weihuang-jedi
weihuang-jedi requested a review from a team as a code owner July 1, 2026 15:49
@weihuang-jedi
weihuang-jedi marked this pull request as draft July 1, 2026 15:49

@maddenp-cu maddenp-cu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As with #196, this PR does not reference an open Issue, does not integrate with the repository's existing structure and code-quality testing, and I'm not aware of any discussion about the need for such a feature or how best to implement it.

@AnilKumar-NOAA

Copy link
Copy Markdown
Contributor

As with #196, this PR does not reference an open Issue, does not integrate with the repository's existing structure and code-quality testing, and I'm not aware of any discussion about the need for such a feature or how best to implement it.

Yes. Agreed with @maddenp-cu. @weihuang-jedi Please don't create PR for verification features plots yet. Lets keep it as it is for now. We will integrate our verification scorecard in the eagle-tools framework at some point in near term.

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