-
Notifications
You must be signed in to change notification settings - Fork 3
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
pyaqsapi.bbox.monitors() #7
Comments
@gregorywanderson, so that I can better understand your issue can you please give me some more deatils as to what you mean that this function is not returning the right results? What do you think should be the correct results? |
The bounding box captures Lake Michigan and the surrounding shoreline. When
I used the UV parameter 63302 it gives UV monitors which I know are in the
bounding box. When I use the ozone parameter 44201 it gives returns sites
outside of the bounding box e.g. Alabama, Florida and Kentucky in addition
to sites inside of the bounding box.
…On Tue, Nov 21, 2023 at 12:04 PM mccroweyclinton-EPA < ***@***.***> wrote:
@gregorywanderson <https://github.com/gregorywanderson>, so that I can
better understand your issue can you please give me some more deatils as to
what you mean that this function is not returning the right results? What
do you think should be the correct reseults?
—
Reply to this email directly, view it on GitHub
<#7 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APW4TVK4MKKXASH76C4LNADYFTUJPAVCNFSM6AAAAAA7TVOFSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRRGQYDOMBYGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thank you for the clarification, I see the discrepency and will work with the EPA AQS DataMart team investigate this. |
@gregorywanderson, there was a small bug in the code. Hopefully this latest commit fixes the issue that you are seeing. Please pull the latest commit and let me know if you are still having this issue. Thank you |
Issue is addressed in commit a39ff6c. |
I got this to work successfully for MET parameters like UV 63302 but it does not seem to return the right results for CRITERIA parameters like ozone 44201, ... for a bounding box around Lake Michigan
lm_o3_monitors = aqs.bybox.monitors(parameter="44201",#"63302",#"88101","44201","42602","88101"
bdate=dt.datetime(2020,1,1),
edate=dt.datetime(2020,12,31),
minlat="41",
maxlat="46.5",
minlon="-89",
maxlon="-85")
display(lm_o3_monitors)
The text was updated successfully, but these errors were encountered: