Skip to content

Framework for 1D elevation change calibration#137

Merged
btobers merged 45 commits intodevfrom
130_elev_change_calib
Oct 16, 2025
Merged

Framework for 1D elevation change calibration#137
btobers merged 45 commits intodevfrom
130_elev_change_calib

Conversation

@btobers
Copy link
Copy Markdown
Collaborator

@btobers btobers commented Oct 7, 2025

No description provided.

@btobers btobers linked an issue Oct 8, 2025 that may be closed by this pull request
@btobers btobers linked an issue Oct 9, 2025 that may be closed by this pull request
@btobers btobers marked this pull request as ready for review October 14, 2025 19:41
@btobers btobers requested a review from drounce October 14, 2025 19:41
@btobers
Copy link
Copy Markdown
Collaborator Author

btobers commented Oct 14, 2025

@drounce I'm marking this PR as ready to review. All tests are passing, but there is no test in place as of now to calibrate against 1d elevation change data, which of course is the main addition of this PR. I need to think about if/how we might do this - perhaps a very short chain could be run in a demo notebook, since this calibration method is much slower than our previous calibration options.

@btobers btobers removed the request for review from drounce October 14, 2025 19:46
@btobers
Copy link
Copy Markdown
Collaborator Author

btobers commented Oct 14, 2025

Actually, maybe I should just do this before closing this PR.

@btobers btobers marked this pull request as draft October 14, 2025 19:46
@btobers
Copy link
Copy Markdown
Collaborator Author

btobers commented Oct 15, 2025

@drounce this is now ready for review! I added a new notebook (https://github.com/PyGEM-Community/PyGEM-notebooks/blob/dev/advanced_test_spinup_elev_change_calib.ipynb) that tests inversion, spinup, and then calibration against elevation change data for Gulkana, running a very short chain just to demonstrate functionality. All tests passing, which now means that the this new functionality is working as expected. Adding this workflow to the test suite does slow things down a little bit...We're now at ~12 minutes to run all tests.

I also updated the sample dataset we have on Drive to include data for Gulkana for the purposes explained here.

@btobers btobers marked this pull request as ready for review October 15, 2025 17:28
@btobers btobers requested a review from drounce October 15, 2025 17:32
@btobers btobers self-assigned this Oct 15, 2025
@btobers btobers requested a review from albinwwells October 16, 2025 13:17
@btobers btobers merged commit fd6a26c into dev Oct 16, 2025
1 check passed
@btobers btobers deleted the 130_elev_change_calib branch October 16, 2025 14:21
joh-brunner added a commit to GLACMASS/PyGEM that referenced this pull request Oct 20, 2025
Framework for 1D elevation change calibration (PyGEM-Community#137)
@btobers btobers mentioned this pull request Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Elevation change calibration Introduce a plot module

2 participants