-
Notifications
You must be signed in to change notification settings - Fork 265
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
ADD: Addition of above_toa_filter. #1586
Conversation
This filter replaces toa parameter in gridding code and allows for the user to specify their own remove above toa filter by using the gatefilter for consistency between both gridding functions.
@mgrover1 Question is, do we continue if gatefilter is None, remove anything above 17000.0 for gridmapper, or leave it fully up to the user. It was originally in to speed up the slower method of grid_mapper |
@mgrover1 Changes have been done for backwards compatibility, this should be good now for a final review. |
@mgrover1 When you get a chance, this is good for a final review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we keep the toa parameter in map_gates_to_grid
, and remove when we release in 2.0?
@mgrover1 The only reason I removed it from the map_gates_to_grid but not grid mapper is that in map_gates_to_grid I did not see the parameter being used anywhere when I search for toa so I believe it was just there. I can add it back to be safe. |
@mgrover1 Added it back in. |
the failing test is fixed in #1597 |
thanks for making those changes @zssherman |
This filter replaces toa parameter in gridding code and allows for the user to specify their own remove above toa filter by using the gatefilter for consistency between both gridding functions.