search

NOAA-GSL / pygraf

ADB's Python graphics package
MIT License
10 stars 11 forks source link

Add a model-name dependence for some cloud fields #236

Closed jaymes-kenyon closed 7 months ago

jaymes-kenyon commented 7 months ago

This update uses the model name to correctly assign handles for the ceiling (exp-2) and cloud-base height fields.

jaymes-kenyon commented 7 months ago

Hi @cshartsough — The main rationale for this PR is the ceiling-exp2 field. A previous PR changed the handle for this field to work with RRFS output, but the same PR inadvertently caused HRRR graphics to stop for this field. With this update, hopefully we will see the ceiling-exp2 plots from both the RRFS and HRRR (each using different handles for the same field).

cshartsough commented 7 months ago

That's good, I think it should do the trick nicely.

jaymes-kenyon commented 7 months ago

I am going to let this run in a RRFS retro overnight, just as a quick test. I will follow-up once I have some successful plots.

jaymes-kenyon commented 7 months ago

@cshartsough @Brian-Jamison — My tests have worked, so I think this PR is ready for merging.