Skip to content

Visualize multiple glaciers #2

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
Open
afisc wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Visualize multiple glaciers #2

afisc wants to merge 18 commits into from

Conversation

@afisc
Copy link
Owner

@afisc afisc commented Nov 28, 2023

PR for discussion with Patrick

afisc and others added 18 commits November 27, 2023 15:06
@afisc
moved combine_grids function from graphics to utils
a4a1e01
@afisc
introduced 'combine_distributed_thickness' functionality - not finish…
816cc25 
…ed yet
@afisc
split up and redesigned the 'define_glacier_region' to also use parts…
2296a14 
… of it for the new function 'dem_for_combined_grid'
@afisc
bugfix
874b2d0
@afisc
bugfix
f764bab
@afisc
added comments to the combine_distributed_thickness function
628ef97
@pat-schmitt
cleaning and refractoring
83781d4
@afisc
added 'unwrapped' option to the entity_task decorator function
6eedc1a
@afisc
worked on 'combined_distributed_thickness'
0711500
@afisc
adapted read_geotiff_dem
df70124
@afisc
spelling mistake
44658d6
@afisc
adapted process_dem
32b0a2f
@afisc
now using the correct data (gridded_simulation)
1a1be8e
@afisc
creating gridded_data.nc for combined grid with file ending
09cb831
@afisc
Merge remote-tracking branch 'upstream/master' into visualize_multipl…
55a2971 
…e_glaciers

# Conflicts:
#	oggm/core/gis.py
@afisc
merged RGI7 related changes with 'multiple glacier visualization' cod…
b63a722 
…e changes
@afisc
adapted combine_distributed_thickness
eda6a10
@afisc
Merge remote-tracking branch 'upstream/master' into visualize_multipl…
ffbcca2 
…e_glaciers
@coveralls
Copy link

coveralls commented Dec 21, 2023

Pull Request Test Coverage Report for Build 7293527404

  • 342 of 478 (71.55%) changed or added relevant lines in 15 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on visualize_multiple_glaciers at 85.107%
Changes Missing Coverage Covered Lines Changed/Added Lines %
oggm/core/centerlines.py 10 11 90.91%
oggm/shop/rgitopo.py 1 2 50.0%
oggm/utils/_funcs.py 37 38 97.37%
oggm/graphics.py 3 5 60.0%
oggm/utils/_workflow.py 65 67 97.01%
oggm/workflow.py 6 10 60.0%
oggm/shop/glathida.py 67 73 91.78%
oggm/shop/millan22.py 13 19 68.42%
oggm/cli/prepro_levels.py 15 37 40.54%
oggm/core/gis.py 71 98 72.45%
Totals Coverage Status
Change from base Build 7016726729: 85.1%
Covered Lines: 11589
Relevant Lines: 13617
💛 - Coveralls

afisc added a commit that referenced this pull request Jul 3, 2024
@afisc
adapted tutorial script to create visualizations #2
d069bff
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.

4 participants

@afisc @coveralls @pat-schmitt