Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

restrict post computation from undefined points #240

Closed
junwang-noaa opened this issue Dec 11, 2020 · 10 comments · Fixed by #306
Closed

restrict post computation from undefined points #240

junwang-noaa opened this issue Dec 11, 2020 · 10 comments · Fixed by #306
Assignees
Labels
enhancement New feature or request

Comments

@junwang-noaa
Copy link
Contributor

Currently in POST, there is no restriction to keep the computation only happening on valid points with reasonable value. The undefined value are participating in the computation in many places which causes wrong value in post products. One example is when regional output domain is larger than the forecast computational domain. FillValue is put on the extra domain for all the fields, however those fields are still involved in post computation and causes issues.

@WenMeng-NOAA WenMeng-NOAA added the enhancement New feature or request label Dec 30, 2020
@HuiyaChuang-NOAA
Copy link
Collaborator

After Wen tested and merged Jun's UPP updates, Jesse will take over and scan through all UPP subroutines to make sure derivations are only done when none of input has missing values.
Jesse will also follow Jun's advice to test use of negative value as missing value instead, e.g., -1.e10
Jesse's task is tracked and documented in
https://vlab.ncep.noaa.gov/redmine/issues/87352

@JesseMeng-NOAA
Copy link
Contributor

JesseMeng-NOAA commented Jan 28, 2021 via email

@WenMeng-NOAA
Copy link
Collaborator

@JesseMeng-NOAA The UPP branch was updated with the PR #255 for regional in-line post.

@WenMeng-NOAA
Copy link
Collaborator

@JesseMeng-NOAA Any updates on this issue?

@JesseMeng-NOAA
Copy link
Contributor

@JesseMeng-NOAA Any updates on this issue?

working on it

@JesseMeng-NOAA
Copy link
Contributor

pull request submitted
#306

@WenMeng-NOAA WenMeng-NOAA linked a pull request Apr 29, 2021 that will close this issue
@WenMeng-NOAA
Copy link
Collaborator

On Thu, Apr 29, 2021 at 4:21 PM Wen Meng - NOAA Affiliate wen.meng@noaa.gov wrote:
Jesse,

Would you run the regression tests before and after your changes, and compare all fields in the datasets from all applications to
ensure no fields having all 0 values changes or from all 0 to valid values changes? It seems the utility cmp_grib2_grib2_new might fail to
compare two fields either one with all 0 values.

Thanks,

Wen

@WenMeng-NOAA
Copy link
Collaborator

On Fri, Apr 30, 2021 at 2:47 PM Jesse Meng - NOAA Affiliate jesse.meng@noaa.gov wrote:
All output checked.
The fields show up in *.diff are due to one of the following reasons,

  1. the entire field is undef in both rt base file AND the test file
  2. boundaries of regional models
  3. hrrr and rap pe test, Wen has mentioned it's associated with pending GSD unification
  4. surface ALBEDO, fixed bug in current FIXED.f for uninitialized GRID1

Resolved.

@WenMeng-NOAA
Copy link
Collaborator

On Fri, Apr 30, 2021 at 3:05 PM Wen Meng - NOAA Affiliate wen.meng@noaa.gov wrote:
Jesse,

My understanding is that you have checked all datasets from gfs, gefs, rap, hrrr, hafs, fv3lam. There were no the following
situations:

  1. values in fields are changed from all 0 (undefined or invalid ) into valid.
  2. values in fields are changed from in valid into all 0 (undefined or invalid).

Thanks,

Wen

@WenMeng-NOAA
Copy link
Collaborator

Correct.

Dr. Jesse Meng
IMSG at NOAA/NWS/NCEP/EMC
5830 University Research Ct. Room 2037
College Park, MD 20740
email: jesse.meng@noaa.gov

WenMeng-NOAA pushed a commit that referenced this issue May 3, 2021
* 20210414 Jesse Meng commit progress for Issue #240 restrict post computation from undefined points

* 20210416 Jesse Meng commit progress for "restrict post computation from undefined points #240"

* 20210422 Jesse Meng commit progress for #240 restrict post computation from undefined points

* 20210423 Jesse Meng fix bug in CALVIS_GSD PR#306

* 20210427 Jesse Meng commit EMC_post issue #240 restrict post computation from undefined points

* 20210429 Jesse Meng fix instantaneous surface SW in CLDRAD.f

Co-authored-by: wx22mj <Jesse.Meng@v71a1.ncep.noaa.gov>
Co-authored-by: wx22mj <Jesse.Meng@v71a2.ncep.noaa.gov>
Co-authored-by: wx22mj <Jesse.Meng@v72a1.ncep.noaa.gov>
EricJames-NOAA pushed a commit to EricJames-NOAA/UPP that referenced this issue Dec 14, 2022
* Adding docs for Rocoto Jinja template.

* Changing the docstring to a Jinja comment.

* Cleaning up some white space.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants