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

Initial DamageCurve implementation plus.. #55

Merged
merged 23 commits into from
Aug 3, 2023
Merged

Initial DamageCurve implementation plus.. #55

merged 23 commits into from
Aug 3, 2023

Conversation

tomalrussell
Copy link
Member

Features:

  • PiecewiseLinearDamageCurve implementation and read from CSV/XLSX

Refactoring:

  • update split_polygons for performance improvement
  • renaming Transform to GridDefinition
  • snail.io and snail.intersection separation of concerns, e.g. with read_raster_band_data and get_raster_values_for_splits
  • fixes to tutorial notebooks to keep up-to-date

@tomalrussell
Create damage curves
3cc1f9a 
- damage curve objects intended to encode commonly-implemented
  details for direct damage assessment
- implementation of piecewise-linear damage curves
- methods to apply curve to exposure, scale/translate curves
@tomalrussell
Update tutorials to use snail intersection and damage curves
3bda1e6
@tomalrussell
Refactor tests
dba4403 
- drop unittest.TestCase inheritance
- parameterize to avoid some tests-in-loops
- assert .all() on lists of bools
- reorder list of polygon split rings.. should be order-independent?
@tomalrussell
No specific order required from split polygons
2d69f61 
- in test, sort along a Hilbert curve using each split centroid
- test 'actual.equals(expected)' to ignore specific vertex ordering
@tomalrussell
Rename PiecewiseLinearDamageCurve and GridDefinition
1b1a185
@tomalrussell
Rename grid variables
d64a401
@tomalrussell
Add test/tutorial packages
e2e0015
@tomalrussell
Use importlib for version
50dde67 
pkg_resources is deprecated.
@tomalrussell
Separate reading rasters from getting values
9e788c4
@tomalrussell
Rename t/transform variables to grid/s
bc8d2af
@tomalrussell
Read a curve from CSV or Excel file
60a7d7f
@tomalrussell
Read grid definition from raster or extent
83c0b07
@tomalrussell
Attempt at multiprocessing
5b1c011
@tomalrussell
Rewrite polygon intersection as numpy/vectorised
28e691f
@tomalrussell
Rename t to grid
66e49f9
@tomalrussell
Use generic split_features_for_rasters in "split"
8562245 
- rename transform:grid
- add INFO logging
- use list-of-rasters wrapper function
@tomalrussell
Default to pyogrio engine for reading to geopandas
af54a42
@tomalrussell
Add contextily for tutorial map backgrounds
c1c497f
@tomalrussell
Update tutorials
b8502d0
@tomalrussell
Format
60fa4a7
@tomalrussell
Update TODOs
ae3a76f
@tomalrussell
Update type hinting
c48cb35
This was referenced Jul 28, 2023
Closed
Open
@tomalrussell tomalrussell merged commit e1bfe95 into main Aug 3, 2023
9 checks passed
@tomalrussell tomalrussell mentioned this pull request Aug 9, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thomas-fred thomas-fred Awaiting requested review from thomas-fred

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@tomalrussell