Skip to content
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

Chore/54 test coverage #55

Merged
merged 19 commits into from
Jan 28, 2025
Merged

Chore/54 test coverage #55

merged 19 commits into from
Jan 28, 2025

Conversation

chrisbc
Copy link
Member

@chrisbc chrisbc commented Jan 16, 2025

100% coverage of the main packages to provide better stability (#54)

  • solvis.solution.inversion_solution package
  • solvis.solution.fault_system_solution package
  • solvis.solution.* modules
  • solvis.filter package
  • also forced a few dangling issues to be revisited .... mainly around serialisation.

@chrisbc chrisbc force-pushed the chore/54_test_coverage branch from ffedcf2 to 28b7d51 Compare January 21, 2025 01:40
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.31%. Comparing base (cd50f24) to head (52c7006).
Report is 3 commits behind head on pre-release.

Additional details and impacted files
@@               Coverage Diff               @@
##           pre-release      #55      +/-   ##
===============================================
+ Coverage        91.57%   99.31%   +7.74%     
===============================================
  Files               22       22              
  Lines             1305     1313       +8     
  Branches           114       67      -47     
===============================================
+ Hits              1195     1304     +109     
+ Misses              86        8      -78     
+ Partials            24        1      -23     
Flag Coverage Δ
unittests 99.31% <100.00%> (+7.74%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@chrisbc chrisbc marked this pull request as ready for review January 23, 2025 01:00
@chrisbc chrisbc requested review from voj and chrisdicaprio January 23, 2025 01:00
Copy link
Collaborator

@voj voj left a comment

Choose a reason for hiding this comment

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

Looks good, minor comments

@chrisbc chrisbc mentioned this pull request Jan 27, 2025
Copy link
Collaborator

@chrisdicaprio chrisdicaprio left a comment

Choose a reason for hiding this comment

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

looks good @chrisbc , just 2 small changes

@chrisbc chrisbc merged commit c642ce7 into pre-release Jan 28, 2025
11 checks passed
@chrisbc chrisbc deleted the chore/54_test_coverage branch January 28, 2025 04:46
chrisbc added a commit that referenced this pull request Feb 27, 2025
* new fault_system_solution_helper module
* a simple rupture grouping algo;
* drop_zero_rate differentiates between FSS and IS;
* refactor subsection filter & tests
* migrate to filter_rupture_ids.for_subsections();
* migrate to filter_rupture_ids.for_parent_fault_ids();
* fixed method names; added doc page for subsection_id_filter;
* refactor filter tests; WIP on DeprecationWarnings; new filter_parent_fault_ids; detox
* fixing many deprecation warnings; updated geopandas, pandas, numpy, pyvista libs; changes to read_csv_dtype configuration; detox
* refactor filters int package; more test cover; implement new filters; migrate old functions and deprecate warn the originals;
* unifying filter methods; improve docstrings & tests;
* implement rate and mag filtering on rupture_id_filter;
* migrate find rupture polygons into filter class; move drop_zero_rates into class init;
* implement ChainableSetBase; migrate FilterRuptureIds to be chainable;
* add set operation support for chaining (join_prior) and test coverage;
* join arguments tidied up; strings and ENUMS are supported;
* fix set operand method names;
* fix set operations return type
* implement set operands for supported set methods; improve docs
* merge 3d geometry changes from @voj; add some testing;
* new participation rate functions; detox
* test 3d surfaces
* migrate participation functions into classes proper; add FSS section_participation;
* make subsection_id_filter chainable;
* added participation methods to fault_system_solution
* simplify FSS participation using rate_weighted_mean; refactor dataframe dtypes; participation performance testing;
* standardise participation functions API and return column;
* fix rate_column; add .all() methods to filter classes;
* add TinyInversionSolution fixture; tune pytest fixture scoping for massive test speed up (~3 times faster);
* doc and docstrings improvements; Better typing in filter package;
* more on rupture_grouping
* add module docstring to inversion_solution_file;
* adding dataframe schema validations with panderas;
* WIP, adding InversionSolutionFile::RuptureRateSchema;
* Feature/52 refactor inversion solution (#53)
* Chore/54 test coverage (#55)
* Chore/59 doc fixes (#61)
* make parent fault filter chainable (#66)
* fix for #57 (return int type for polygons);
* update pytz dep (#69)
* Feature/49 named fault participation (#67)
* Feature: named_fault_participation_rate #49
* Epic Feature: enhanced participation rate options #42
* refactor to remove *Protocol superclasses (#73)
* refactor to remove *Protocol superclasses; migrate and improve docstrings; detox
* Feature/71 scale rupture rates (#74)
* fold in changes suggested by @chrisdicaprio on PR #73;
* fix drop_zero_rates on FilterRuptureIds.for_magnitude(); 
* fix for_parent_fault_ids
* fix test_ruptures_for_subsections
* fix for_rupture_rate
* fix for_polygon ; fix impacted tests;
* Bump version: 0.13.0 → 1.0.0-alpha
* add joint_type arg to relevant FilterRuptureId class methods (#80)
* add joint_type arg to `for_parent_fault_names` *  `for_parent_fault_ids`; add tolist() convenience method;
* refactor for readability/efficiency; add full test coverage for new arg;
* add `join_type` arg to `for_subsection_ids` (working but a little slow); detox;
* rename arg for consistency
* refactoring; new docstrings; named_faults support; detox;
* remove commented line
* changelog entry;
* Bump version: 1.0.0-alpha → 1.0.0-beta
* align version ids
* Chore/82 make docs and api align (#83)
* Bump version: 1.0.0-beta-0 → 1.0.0

---------

Co-authored-by: Chris DiCaprio <[email protected]>
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.

3 participants