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

Issue with intensity jump when resampling distributions #416

Closed
dnerini opened this issue Aug 5, 2024 · 3 comments
Closed

Issue with intensity jump when resampling distributions #416

dnerini opened this issue Aug 5, 2024 · 3 comments
Assignees

Comments

@dnerini
Copy link
Member

dnerini commented Aug 5, 2024

The main issues for me is indeed the intensity jump. Could this be because the NWP has very high intensities, but as these are not present in the blended forecast, the latter is unrealistically scaled up overall?
Do you suggest not filling anything outside the radar mask (so we have nans in the final forecast)? Or just for the resampling?

Originally posted by @ladc in #390 (comment)

@dnerini
Copy link
Member Author

dnerini commented Aug 5, 2024

I'm opening a new issue here to better track the problem described by @ladc in #390.

@RubenImhoff did I understand correctly that you're working on it?

@RubenImhoff
Copy link
Contributor

Great, thanks, @dnerini! I'm not actively working on it, but happy to make some code changes. I think I prefer to have data outside the radar mask, but not filling anything outside the radar mask for resampling seems the way to go. What do you think?

@ladc
Copy link
Contributor

ladc commented Aug 27, 2024

Should be fixed in #428.

@ladc ladc closed this as completed Aug 27, 2024
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

No branches or pull requests

3 participants