-
Notifications
You must be signed in to change notification settings - Fork 33
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
Region support only works with obs Dataframes #278
Comments
The domains being based on a query is still useful though. Because the surface observations normally have meta data to separate out the domains, so I do not want to change this feature. As you can easily plot based on state, city, etc. this way. The boxes in MONET and MONETIO I doubt are as accurate as they would be based on lat lon boxes that may not perfectly represent the domain. But adding the capability to use domains for data that does not provide meta data for this would be useful. But we need to do it in a way that does not break the query feature for surface observations. Can we add a yaml true/false statement for this? Something like domain_query = True (default and domains are based on the meta data in the observation file) and if you set this to False, then it uses the boxes defined in MONET and MONETIO to do this query. What do you think about this? |
That is a good solution. I am happy to move forward with that, or at least adding an extra check for the satdata. |
Great thanks! |
Closing this as it has been solved in #280 |
Dear all,
Currently, the region support that works with AirNow does not work for everything else the way it should.
The code is reading regions from the information already provided by the obs DataFrame (the driver does a query. However, with EPA and Giorgi regions there is no reason for that: we have boxes defined in MONET, that MONETIO can import.
The sat data with the current setup just wouldn't work for any domain_type different from 'all' (I don't know about aircraft).
If @rschwant and @rrbuchholz agree, I suggest changing it (for now, I could stick to the satdata for that) so that it can subset the domain, at least in the case of EPA and Giorgi regions. Adding arbitrary square boxes should be close to no extra effort.
We can mention this in the tutorial as WIP.
Let me know your thoughts
Pablo
The text was updated successfully, but these errors were encountered: