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

NFFD doesn't match ClimateNA #295

Open
cmahony opened this issue Aug 16, 2024 · 4 comments
Open

NFFD doesn't match ClimateNA #295

cmahony opened this issue Aug 16, 2024 · 4 comments
Labels
bug Something isn't working v1.0.0 complete before release of version 1.0.0

Comments

@cmahony
Copy link
Collaborator

cmahony commented Aug 16, 2024

NFFD not matching between climr and ClimateNA. at an annual level, it seems to be related to longitude
climrVclimna NFFD

on a monthly level it is confined to latitudinal regions
climrVclimna NFFD_03
climrVclimna NFFD_10

@cmahony cmahony added bug Something isn't working v0.2.0 complete for next minor release labels Aug 16, 2024
@cmahony
Copy link
Collaborator Author

cmahony commented Aug 16, 2024

@cmahony
Copy link
Collaborator Author

cmahony commented Aug 19, 2024

I've taken a look and haven't found an error in our code. The only associated variable is Tmin, and there are no discrepancies.

one key thing is that the NFFD parameters published in Wang (2016) are not what is used in ClimateNA. Tongli provided us with alternative parameters, see #68. perhaps importantly, those equations are for "NFFD_wna", suggesting they were originally calibrated for WNA. it is outside the ClimateWNA domain that we are seeing this discrepancy in NFFD. it seems likely that Tongli may have updated the parameters for eastern North America.

@cmahony cmahony added v1.0.0 complete before release of version 1.0.0 and removed v0.2.0 complete for next minor release labels Aug 19, 2024
@cmahony
Copy link
Collaborator Author

cmahony commented Aug 19, 2024

downgrading priority to V1, and entering as a known issue. we'll get in touch with tongli to see if there are updated parameters for ENA

@cmahony
Copy link
Collaborator Author

cmahony commented Aug 19, 2024

Note that PAS has a similar pattern. PAS was the other variable set that tongli provided us wiht WNA parameters for.
climrVclimna PAS_03
climrVclimna PAS_10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v1.0.0 complete before release of version 1.0.0
Projects
None yet
Development

No branches or pull requests

1 participant