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

duplicate time issues from original dataset #81

Closed
yunfangsun opened this issue Jun 15, 2023 · 3 comments
Closed

duplicate time issues from original dataset #81

yunfangsun opened this issue Jun 15, 2023 · 3 comments
Assignees
Labels
duplicate This issue or pull request already exists

Comments

@yunfangsun
Copy link

I have used stormevents to download the Hurricane IAN 2022's best track file. Because there are duplicate time slots in the original data sever, the outputs from stormevents keep the duplicate data.

When I try to use the hurricane track file to run PaHM case, it generate errors "InitLogging not called :: ProcessAsymmetricVortexData: 4 isotachs were nonzero." the details could be found at PaHM issue 26.

Is there a way to normalize the stormevents or PaHM scripts to avoid such kind of errors?
@SorooshMani-NOAA @pvelissariou1

@SorooshMani-NOAA SorooshMani-NOAA self-assigned this Jun 15, 2023
@SorooshMani-NOAA
Copy link
Collaborator

Thanks for creating this issue. As we discussed during the meeting and on noaa-ocs-modeling/PaHM#26, the source of the issue is the original data from ATCF file. We need to add explicit normalization methods/functions in stormevents.

Right now AFAIK there's an implicit normalization for OFCL advisory to fix eye and background pressure.

I was thinking about adding a method like normalize_for_parametric_models() method to the VortexTrack class so that one can explicitly ask for the normalization such as removing duplicates for a given forecast time.

I also think ideally the fix for pressures (which happens implicitly right now) need to be explicit rather than implicit. What do you think @WPringle

@WPringle
Copy link
Contributor

@SorooshMani-NOAA Doing some cleaning up explicitly could make sense I agree.

@SorooshMani-NOAA
Copy link
Collaborator

Close as duplicate!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants