Skip to content

v0.6.1 release branch #185

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

Merged
merged 32 commits into from
Dec 12, 2022
Merged

v0.6.1 release branch #185

merged 32 commits into from
Dec 12, 2022

Conversation

wsavran
Copy link
Collaborator

@wsavran wsavran commented Feb 10, 2022

No description provided.

@wsavran wsavran changed the title Update setup.py v0.6.1 release branch Feb 10, 2022
khawajasim and others added 7 commits February 23, 2022 08:37
* quadtree csv reader
* included depth columns in reader
* reader name change
* cartesian_grid.bounds added
* check for zero area
* Added new functions that compute and plot 95% and 97.5% confidence ranges, as well as p values for CSEP consistency tests.
added proper imports for plot_pvalues_and_intervals
minor code formatting
fixed issue incorrect percentile in plot_pvalues_and_intervals
return ax


def plot_pvalues_and_intervals(test_results, ax, var=None):
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bayonato89 i had to make some modifications here to work within pycsep. can you test this new function out and make sure that it's working properly?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does work properly @wsavran

minor code formatting updates
update tests for python 3.10: failing absolute assertion changing to numpy.testing.all_close in csep1 t-test
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2022

Codecov Report

Base: 55.84% // Head: 54.55% // Decreases project coverage by -1.28% ⚠️

Coverage data is based on head (09e2769) compared to base (ad217ed).
Patch coverage: 27.33% of modified lines in pull request are covered.

❗ Current head 09e2769 differs from pull request most recent head 76844eb. Consider uploading reports for the commit 76844eb to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   55.84%   54.55%   -1.29%     
==========================================
  Files          20       20              
  Lines        3560     3668     +108     
  Branches      568      540      -28     
==========================================
+ Hits         1988     2001      +13     
- Misses       1452     1550      +98     
+ Partials      120      117       -3     
Impacted Files Coverage Δ
csep/core/binomial_evaluations.py 14.48% <14.48%> (ø)
csep/utils/readers.py 46.44% <20.00%> (-1.39%) ⬇️
csep/utils/comcat.py 83.74% <27.77%> (-2.85%) ⬇️
csep/core/regions.py 47.04% <38.88%> (+0.32%) ⬆️
csep/core/catalogs.py 60.95% <60.00%> (-0.15%) ⬇️
csep/core/forecasts.py 57.57% <73.33%> (+0.18%) ⬆️
csep/utils/calc.py 54.78% <85.71%> (ø)
csep/utils/basic_types.py 89.28% <100.00%> (ø)
csep/__init__.py

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

bayonato89 and others added 13 commits August 18, 2022 11:01
* Changed np for numpy in the poisson_evaluations.py file
* Added non-Poissonian comparative and consistency tests.
* format  docstrings
* Rename non_poissonian_evaluations.py to binomial_evaluations.py
* Update binomial_evaluations.py
removed unused imports

Co-authored-by: wsavran <35315438+wsavran@users.noreply.github.com>
Added NZ collection and testing regions, and parser to csep region object.
Added test for NZ testing regions
fix: region border plot. Now uses shapely that merges all cells and gets the boundary of the total region. Bypasses floating point issues by rounding down to 2 decimals
fix: load_ascii now creates cell polygons clockwise
fix: tight_bbox has option for precision. It is not necessary for lat_lon, but explicited the argument, in case cartesian coordinates were implemented.
build: added shapely as requirement
* Added some documentation on new binary CSEP tests.
added proper imports for plot_pvalues_and_intervals
minor code formatting
fixed issue incorrect percentile in plot_pvalues_and_intervals
minor code formatting updates
update tests for python 3.10: failing absolute assertion changing to numpy.testing.all_close in csep1 t-test
* Changed np for numpy in the poisson_evaluations.py file
* Added non-Poissonian comparative and consistency tests.
* format  docstrings
* Rename non_poissonian_evaluations.py to binomial_evaluations.py
* Update binomial_evaluations.py
removed unused imports

Co-authored-by: wsavran <35315438+wsavran@users.noreply.github.com>
* Added some documentation on new binary CSEP tests.
@wsavran
Copy link
Collaborator Author

wsavran commented Sep 23, 2022

@khawajasim, @bayonato89, I'm writing some docs and reviewing the binomial_tests.py, and i have a couple of questions.

  1. It seems we are missing the function _simulate_catalog(). is this from poisson_evaluations.py? if not, can you add the binomial _simulate_catalog() to this pull request?
  2. This also makes me think we should have a test case for this function. Can you give me a worked example that we could implement like this, for one of the binomial tests with simulation?

@bayonato89
Copy link
Contributor

Just added the _simulate_catalog function (although I think I sent it to the wrong branch). Let me know if I should resubmit @wsavran. The function was written by @khawajasim. We should be careful when calling it, because the _simulate_catalog functions for Poisson and non Poisson consistency tests share the same name.

pabloitu and others added 2 commits November 14, 2022 10:59
* feat: Added query_bsi, based on comcat query functions. Can be called 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.
* docs: added query_bsi to api reference and mentioned it in tutorial/catalog_filtering
styl: modified query_comcat docstring
* fix: added maxdepth to query_comcat and query_bsi wrappers at top and bottom level. default=1000. Maybe these functions should have the arguments to completely define an experiment catalog (perhaps just max_mag and its ready)
* added to unit test for binary_joint_LL
bayonato89 and others added 8 commits November 14, 2022 11:55
* Added non-Poissonian comparative and consistency tests.
* Added a _simulate_catalog function for binary likelihood tests

Co-authored-by: wsavran <35315438+wsavran@users.noreply.github.com>
* style: formatted function plot_consistency_test to use original dependencies aliases
* fix: changed errorbar to be defined from the test distribution mean.
* feat: added option to plot test distribution's mean.
* styl: homogenized plot_poisson_consistency_test and plot_consistency_test
* build: removed matplotlib pin
style: changed variable names in simulation function to keep it consistent
style: updated named of tests in test_evaluations.py to better reflect the test
Added verbose option to get_event_counts()
Added additional test case to CatalogForecast creation
style: changed names of functions in test_evaluations.py
* testing_likelihood
* name correction
* fixing imports
@wsavran wsavran merged commit 48bddff into master Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants