feat: Added query_bsi #201
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… from csep.__init__, wrapping the readers._query_bsi. This is based in csep.utils.comcat.search functions. mods: (i) Exception catchs for Hostname mismatch in SSL verification (the bsi does not go through https). (ii) in comcat and bsi, ids are set different. SummaryEvent now handles this with try/except. (iii) bsi returns datetimes as utc strings, instead of timestamp as comcat.
…atalog_filtering styl: modified query_comcat docstring
Codecov ReportBase: 55.73% // Head: 55.17% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #201 +/- ##
==========================================
- Coverage 55.73% 55.17% -0.56%
==========================================
Files 20 20
Lines 3594 3652 +58
Branches 580 589 +9
==========================================
+ Hits 2003 2015 +12
- Misses 1466 1513 +47
+ Partials 125 124 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
… bottom level. default=1000. Maybe these functions should have the arguments to completely define an experiment catalog (perhaps just max_mag and its ready)
|
cool, will include this in the v0.6.1 release! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
created function to query the Bollettino Sismico Italiano (BSI)
webservices.rm.ingv.it. It is modified from the query comcat functions. Minor mods were done to cat_query functions to handle the different properties returned from bsi compared to comcat (e.g. event['id'] in comcat, event['properties']['eventId'] in bsi).pyCSEP Pull Request Checklist
Type of change:
Please delete options that are not relevant.
Checklist: