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

Improvements for use in flood modelling #231

Closed
3 of 4 tasks
rosepearson opened this issue Jan 17, 2024 · 2 comments · Fixed by #251
Closed
3 of 4 tasks

Improvements for use in flood modelling #231

rosepearson opened this issue Jan 17, 2024 · 2 comments · Fixed by #251
Milestone

Comments

@rosepearson
Copy link
Owner

rosepearson commented Jan 17, 2024

Trace-back when clearing the cached files created to stabilise memory usage during parallel computing.

Line 950 processor.py cached_file.unlink()

https://github.com/rosepearson/GeoFabrics/blob/main/src/geofabrics/processor.py#L950

Also update the windows yml files to be cleaner and remove the spyder dependency.

@rosepearson
Copy link
Owner Author

rosepearson commented Jan 23, 2024

Other suggestions @RegMeasures

  1. Add ability for a raster elevation or roughness patch to be added at the end of the dem/roughness stages (e.g. after the LiDAR is added).
  2. Add support for a stop-bank stage
    i. support z-height polyline
    ii. support crest height from LiDAR along polyline withou z-heights
  • Add a source to the waterways stage so it is clear OSM is the current only source.
  • Support saving out as GeoTiffs (each layer as a separate GeoTiff)

@rosepearson rosepearson changed the title unlink trace in windows environment Improvements for use in flood modelling Feb 22, 2024
@rosepearson rosepearson linked a pull request Jun 10, 2024 that will close this issue
16 tasks
@rosepearson
Copy link
Owner Author

could look at replacing the LFS caching in the GH actions with https://github.com/marketplace/actions/cached-lfs-checkout

@rosepearson rosepearson added this to the 1.1.15 milestone Jul 3, 2024
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 a pull request may close this issue.

1 participant