-
Notifications
You must be signed in to change notification settings - Fork 53
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
IBTrACS version 4 #69
Comments
Note WMO version of IBTracs has been deprecated. |
Running TCRM v3.0.1 with IBTrACS V4 falls over due to 1986226N30276 26/08/1986 12:00:00 longitude being beyond the maximum longitude in the grid. _lonLat function in DataProcess.py fails due to dependency on sampleGrid function in Utilities/grid.Py. The call to np.searchsorted on line 259 in grid.py requires a clip/clamp to keep values in range. |
To properly handle the IBTrACS V4 dataset, the dataProcessing code requires tweaking. Positions are generally supplied every 3 hours in the new IBTrACS, but pressure data is only available at 3 hourly intervals for the most recent seasons. When calculating the pressure rates, the _pressureRate function in DataProcess needs to be parsed only the non-missing pressures and associated dt between these obs, or the entire function needs to be re-designed. I have implemented a workaround for this in my code (I am testing the downstream impact on the generated tracks at present), which I'd be happy to share, but I feel it may be a little clunky. |
Did this end up working @KayShelton? |
My my, helloey Tom, been a while since the PCCSP/PACCSAP days :) At a guess, I am going to say yes, as I didn't comment further. I will need to boot up the machine I ran this on to review the code and find any resulting figures I produced. I will get back to you :) |
Hmm, It looks like I tweaked a few bits to make this work, both in the processData function, and in the child function_pressureRate. |
Kay, sorry, didn't recognise the handle! Been a while indeed. Thanks very much, I'll have a look. |
As of March 2019, IBTRACS version 3 is obsolete. Should update code to point at new dataset, check still tests ok, and also consider vendoring, i.e., bundling the data for distribution with tcrm.
The text was updated successfully, but these errors were encountered: