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.

Sign up for GitHub

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

Jump to bottom

Feature/default aggregation options #45

Open
wants to merge 52 commits into
base: pre-release
Choose a base branch
from
Open

Feature/default aggregation options #45

wants to merge 52 commits into from

Conversation

chrisbc
Copy link
Member

@chrisbc chrisbc commented Mar 4, 2025 •
edited
Loading

TODO: rebase after #40 is merged

This PR is preparation for #46

@chrisdicaprio
unify config file and env names
a9aafcf
@chrisdicaprio
documentation WIP
a0c6fb5
@chrisdicaprio
start on api and useage docs
18706a6
@chrisdicaprio
improve local config
f076bdb
@chrisdicaprio
improve error message
40a5302
@chrisdicaprio
usage documentation
67fa601
@chrisdicaprio
clean up demos
d2d55af
@chrisdicaprio
update dev workflow to run on PR to pre-release
ab98b27
@chrisdicaprio
add chagelog
30075f9
@chrisdicaprio
add version and doc string
d4f7f75
@chrisdicaprio
Bump version: 0.3.2 → 0.4.0
1149abd
@chrisdicaprio
update python versions for test workflow
041ebe1
@chrisdicaprio
update ths dep
15b84e5
@chrisdicaprio
reduce reliance on env vars for tests
f03c66c
@chrisdicaprio
use parquet instead of pickle for test fixture
24ba269
@chrisdicaprio
attempt to fix tests in GHA
8daad77
@chrisdicaprio
continue to fix tests
d2ff8d0
@chrisdicaprio
increase test tolerance
0a55068
@chrisdicaprio
still tweaking test
252c2c0
@chrisdicaprio
spot check 4 points in array
38685e1
@chrisdicaprio
use pytest.approx
25b4915
@chrisdicaprio
increase pytest verbosity
48be15a
@chrisdicaprio
increase test tolerance
caeb22d
@chrisdicaprio
fail-fast=false, atol=1e-6
28046fd
@chrisdicaprio
test pd load from parquet
1c2f849
@chrisdicaprio
test that df is read correctly
fab1031
@chrisdicaprio
remove ubuntu-20 from tests
ec34f8e
@chrisdicaprio
linting
c8b2542
@chrisdicaprio
add csv fixture
eb7dfd3
@chrisdicaprio
change test tolerance depending on agg type
53f4fb4
chrisdicaprio and others added 22 commits June 11, 2024 09:46
@chrisdicaprio
linting
4c0a68b
@chrisdicaprio
new weighed quantiles test for large aggregation
c8dfb38
@chrisdicaprio
stable sort
dedeb35
@chrisdicaprio
update end to end with new fixture
bac3402
@chrisdicaprio
fix spelling
3907f79
@chrisbc
use new writer from THS; update locs for Anne;
18412e2
@chrisdicaprio
correct doc string
3705a7c
@chrisdicaprio
update deps for nzhsm_model
51a3f9b
@chrisdicaprio
fix demo
cbfff22
@chrisdicaprio
fix spelling, direct to usage
2c76da7
@chrisdicaprio
update changelog
d306d61
@chrisdicaprio
loosen dependency version restrictions, rebuild lock file
70b5c4e
@chrisdicaprio
remove py3.12 support. pandas 2.0.3 does not have binarires for py3.12
20986ad
@chrisdicaprio
linting
af2d199
@chrisdicaprio
update dependencies
c2ccb56
@chrisdicaprio
fix dependencies
eb608da
@chrisdicaprio
remove flake8-docstrings, to be put back later
ccbcd1f
@chrisbc
use THS enums to validate imts and agg_types; process all if none…
38e5ab7 
… specified;
@chrisbc
update nzshm library versions;
2b9586c
@chrisbc
detox;
076263c
@chrisbc
change mocking to correct target;
086983c
@chrisbc
test tweaked for optinal imts; detox;
84ece27
@codecov Codecov
Copy link

codecov bot commented Mar 4, 2025

Codecov Report

Attention: Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.

Please upload report for BASE (pre-release@1f117a6). Learn more about missing BASE report.

Files with missing lines Patch % Lines
toshi_hazard_post/aggregation_args.py 85.71% 1 Missing and 1 partial ⚠️
toshi_hazard_post/data.py 33.33% 2 Missing ⚠️
Additional details and impacted files 
@@              Coverage Diff               @@
##             pre-release      #45   +/-   ##
==============================================
  Coverage               ?   78.54%           
==============================================
  Files                  ?       11           
  Lines                  ?      634           
  Branches               ?       69           
==============================================
  Hits                   ?      498           
  Misses                 ?      115           
  Partials               ?       21

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrisbc @chrisdicaprio