🔄 synced file(s) with ODINN-SciML/Sleipnir.jl #143
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facusapienza21
force-pushed
the
repo-sync/Sleipnir.jl/default
branch
from
62dced4
to
0d7b3c7
Compare
facusapienza21
changed the title
facusapienza21
force-pushed
the
repo-sync/Sleipnir.jl/default
branch
from
0d7b3c7
to
5a603a8
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #143 +/- ##
=======================================
Coverage 33.41% 33.41%
=======================================
Files 14 14
Lines 1598 1598
=======================================
Hits 534 534
Misses 1064 1064 ☔ View full report in Codecov by Sentry. |
JordiBolibar
approved these changes
Mar 24, 2024
JordiBolibar
added a commit
that referenced
this pull request
May 24, 2024
* UDEs working with experiment with different MB Fixed a few bugs in the UDE workflow. The UDEs work suprisingly well with a different MB model. This means that using surface velocities is really a well proxy, which is robust to changes in surface elevation. * New composite types for parameters created * Updated mass balance model types * New types and structs for new API The basic structure for the new API is ready. Now we need to slowly couple it with the old workflows. * First prototype of new API for predictions First prototype more or less ready. The test script is running, but I need to optimize the way Results are saved in a file. Right now it's way too slow and heavy. We need to trim that down and save it as netCDF. The tests are still under construction to fit the new API. And of course, I still need to design the hardest part, the UDE. * [WIP] Debugging PDE behaviour of new API * First prototype of new API (#125) * UDEs working with experiment with different MB Fixed a few bugs in the UDE workflow. The UDEs work suprisingly well with a different MB model. This means that using surface velocities is really a well proxy, which is robust to changes in surface elevation. * New composite types for parameters created * Updated mass balance model types * New types and structs for new API The basic structure for the new API is ready. Now we need to slowly couple it with the old workflows. * First prototype of new API for predictions First prototype more or less ready. The test script is running, but I need to optimize the way Results are saved in a file. Right now it's way too slow and heavy. We need to trim that down and save it as netCDF. The tests are still under construction to fit the new API. And of course, I still need to design the hardest part, the UDE. * [WIP] Debugging PDE behaviour of new API * Plot scripts updated * New API prototype (#127) * UDEs working with experiment with different MB Fixed a few bugs in the UDE workflow. The UDEs work suprisingly well with a different MB model. This means that using surface velocities is really a well proxy, which is robust to changes in surface elevation. * New composite types for parameters created * Updated mass balance model types * New types and structs for new API The basic structure for the new API is ready. Now we need to slowly couple it with the old workflows. * First prototype of new API for predictions First prototype more or less ready. The test script is running, but I need to optimize the way Results are saved in a file. Right now it's way too slow and heavy. We need to trim that down and save it as netCDF. The tests are still under construction to fit the new API. And of course, I still need to design the hardest part, the UDE. * [WIP] Debugging PDE behaviour of new API * First prototype of new API (#125) * UDEs working with experiment with different MB Fixed a few bugs in the UDE workflow. The UDEs work suprisingly well with a different MB model. This means that using surface velocities is really a well proxy, which is robust to changes in surface elevation. * New composite types for parameters created * Updated mass balance model types * New types and structs for new API The basic structure for the new API is ready. Now we need to slowly couple it with the old workflows. * First prototype of new API for predictions First prototype more or less ready. The test script is running, but I need to optimize the way Results are saved in a file. Right now it's way too slow and heavy. We need to trim that down and save it as netCDF. The tests are still under construction to fit the new API. And of course, I still need to design the hardest part, the UDE. * [WIP] Debugging PDE behaviour of new API * Plot scripts updated * MB model much more efficient I improved the way the climate is handled by the MB model, making it much more efficient without almost any memory allocation. * CompatHelper: bump compat for Polynomials to 4, (keep existing compat) (#131) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: bump compat for Flux to 0.14, (keep existing compat) (#129) Co-authored-by: CompatHelper Julia <[email protected]> Co-authored-by: Jordi Bolibar <[email protected]> * Update publication information * New architecture up to date with latest changes of Sleipnir, Huginn and Muninn All the redundant files have been removed, and now everything is imported from the sublibraries. The workflow works for predictions, but no tests are implemented yet, since they'll have to cover Inversion and FunctionalInversion, which are still to be coded. * 🔄 synced file(s) with ODINN-SciML/Sleipnir.jl (#130) * 🔄 synced local 'environment.yml' with remote 'environment.yml' * 🔄 synced local '.gitignore' with remote '.gitignore' * CompatHelper: bump compat for AbbreviatedStackTraces to 0.2, (keep existing compat) (#132) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: add new compat entry for Downloads at version 1, (keep existing compat) (#133) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: add new compat entry for Statistics at version 1, (keep existing compat) (#134) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: bump compat for Makie to 0.20, (keep existing compat) (#135) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: bump compat for CairoMakie to 0.11, (keep existing compat) (#136) Co-authored-by: CompatHelper Julia <[email protected]> * CI updated with Julia certificates and CI for any branch * Update Huginn version * Dependencies fixed * Huginn updated * CI updated with fixed SSL certificates for Python * Issue with multiprocessing fixed * Multiprocessing for OGGM fixed * Huginn version changed * Bump version and update Huginn * CompatHelper: bump compat for GR to 0.73, (keep existing compat) (#138) Co-authored-by: CompatHelper Julia <[email protected]> * Test mode added for tests * 🔄 synced file(s) with ODINN-SciML/Sleipnir.jl (#143) * 🔄 synced local 'environment.yml' with remote 'environment.yml' * 🔄 synced local '.github/workflows/CI.yml' with remote '.github/workflows/CI.yml' * CompatHelper: bump compat for OptimizationOptimisers to 0.2, (keep existing compat) (#142) Co-authored-by: CompatHelper Julia <[email protected]> * CompatHelper: bump compat for GeoMakie to 0.6, (keep existing compat) (#140) Co-authored-by: CompatHelper Julia <[email protected]> * [WIP] Reverse diff for functional inversions Work in progress commit. Still errors due to problems with mutable structs. We need to decide to go either for ReverseDiff with PreallocationTools or with Enzyme. * Constrain Huginn version * Updated Project.toml * [WIP] Debugging Reverse after merge with ForwardDiff setup Back to the point of debugging where a choice needs to be made between ReverseDiff and Enzyme. Now this aslo includes the changes from the classic inversions using ForwardDiff. --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: CompatHelper Julia <[email protected]> Co-authored-by: Facundo Sapienza <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
synced local file(s) with ODINN-SciML/Sleipnir.jl.
Changed files
environment.ymlwith remoteenvironment.yml.github/workflows/CI.ymlwith remote.github/workflows/CI.ymlThis PR was created automatically by the repo-file-sync-action workflow run #8404401866