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/derived quantities #81

Closed
wants to merge 522 commits into from
Closed

Feature/derived quantities #81

wants to merge 522 commits into from

Conversation

ryan-kipawa
Copy link
Collaborator

No description provided.

Gediminas Kirsanskas and others added 30 commits January 19, 2024 12:39
Merge branch 'main' into feature/merge_res1d
8337799
@ryan-kipawa
Add repr for ResultLocations. Add counts to repr
79721f2
@ryan-kipawa
Add count to ResultLocations html repr, and names + locations properties
348e23d
@ryan-kipawa
Add type hints to ResultLocations quantities, and repr for ResultQuan…
6c1d405 
…tity and ResultQuantityCollection
@ryan-kipawa
Clean Res1D repr, moving detailed info to info() method
3a38d37
@ryan-kipawa
Add a few more docstrings on Res1D and alias res.network for res.resu…
4809b66 
…lt_network
@ryan-kipawa
Separate Res1D info string creation from printing
71093fe
@ryan-kipawa
Update repr tests
69541d4
@ryan-kipawa
Mark core tests as slow and don't run by default, however, run on CI
9276000
@ryan-kipawa
Make .pytest.ini specifically for CI pipeline
7921e67
@ryan-kipawa
Initial custom pandas dataframe accessor implementation
a6fa93d
@ryan-kipawa
Add tests for pandas extensions.
03a8e68
@ryan-kipawa
Add compact_dataframe() helper method
c7cf10a
@ryan-kipawa
Add notebook with examples of working with MIKE IO 1D hierarchical in…
98beb88 
…dices.
@ryan-kipawa
Merge pull request #76 from DHI/feature/improve-repr-and-autcomplete
78da790
@ryan-kipawa
Add docstring for Mikeio1dAccessor
448326c
@ryan-kipawa
Merge pull request #77 from DHI/feature/pandas_dataframe_accessor_ext…
ff752b3 
…ension

Feature/pandas dataframe accessor extension
Add possibility to merge files using info from Res1D object.
3d6b245 
Also add type hints and improve the docstring.
@gedaskir
Merge pull request #75 from DHI/feature/merge_res1d
b9e189e 
Merging of regular and LTS extreme/periodic res1d files
@ryan-kipawa
Add groupby_level helper method and tests.
495bab9
@ryan-kipawa
Make compact_dataframe work even if not all field names are present.
f39f453
@ryan-kipawa
Add ability to read all quantities on ResultLocations
34a8483
@ryan-kipawa
Add ability to read all quantities on single ResultLocation
7387706
@ryan-kipawa
Initial implementation of ResultFrameAggregator
8c93499
@ryan-kipawa
Add quantity aggregation to ResultReacher.to_geopands()
10342c0
@ryan-kipawa
Add quantity aggregation to ResultNodes.to_geopandas()
6191b38
@ryan-kipawa
Add quantity aggregation to ResultCatchments.to_geopandas()
083fc59
@ryan-kipawa
Update notebook to show examples of exporting to geopandas with quant…
6192fb2 
…ities
@ryan-kipawa
Add unit test framework for ResultFrameAggregator
5c5cb0f
@ryan-kipawa
If time is only kwarg provided, make that the default for other axes.
e67aa58
@ryan-kipawa
Add qqmanning as a default derived quantity
333cbc1
@ryan-kipawa
Drop 'derived' level by default when reading derived quantity
e28d00e
@ryan-kipawa
Possibility to include derived quantities when reading result locations
486cae4
@ryan-kipawa
Add possibility for derived quantities with to_geopandas
dc8b317
@ryan-kipawa
Update tests to reflect new defaulit behaviour
334a1eb
@ryan-kipawa
Update derived quantity testing
bb17258
@ryan-kipawa
Add tests for new derived quantities
0dda800
@ryan-kipawa
Use ground level as default when critical level is not set
1b70959
@ryan-kipawa
Test registering example custom derived quantity
3d8a9a5
@ryan-kipawa
Remove functions no longer used in derived_quantity
0d3990c
@ryan-kipawa
Minimal test that derived quantity plots work
b1276c0
@ryan-kipawa
Exclude type checking from test coverage
1cd3c5d
@ryan-kipawa
Add extra tests for derived quantities
35135ef
@ryan-kipawa
Add tests for unregistering custom derived quantities
6bac9e2
@ryan-kipawa
Fix unregister test for derived quantities
acae9ba
@ryan-kipawa
Add tests for reading all quantities for a type of location, includin…
452f275 
…g derived quantities
@ryan-kipawa
Use more explicit argument name in read: include_derived
f4893bf
@ryan-kipawa
Add missing tests for geopandas with result aggregation, as well as w…
b45b51e 
…ith derived
@ryan-kipawa
Fix result nodes and reaches argument per include_derived arg
64bf374
@ryan-kipawa
Run all notebooks
aa6eff3
@ryan-kipawa
Use impl instead of __implementation__
25ef7aa
@ryan-kipawa
Refactor: extract validation logic from class init
997a231
@ryan-kipawa
Refactor: simplify derived quantity long lines
ac6fd40
@ryan-kipawa
Run all notebooks
29b4c20
@ryan-kipawa
Make derived quantity manager non-global
ddb3df6
@ryan-kipawa
Remove DerivedQuantityManager and use simplified approach
3d32718
@ryan-kipawa
Ignore files generates during notebook runs
7357851
@ryan-kipawa ryan-kipawa mentioned this pull request Oct 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gedaskir gedaskir gedaskir left review comments

Assignees

@ryan-kipawa ryan-kipawa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryan-kipawa @gedaskir