Skip to content

Update feature/ufs_dom from develop 2024-03-08#1024

Merged
travissluka merged 4 commits intofeature/ufs_domfrom
feature/ufs_dom_update_from_develop_20240308
Mar 8, 2024
Merged

Update feature/ufs_dom from develop 2024-03-08#1024
travissluka merged 4 commits intofeature/ufs_domfrom
feature/ufs_dom_update_from_develop_20240308

Conversation

@climbfuji
Copy link
Contributor

@climbfuji climbfuji commented Mar 8, 2024

Description

Update feature/ufs_dom from develop 2024-03-08

I had to resolve a few conflicts and make a few changes, most notably:

Issue(s) addressed

ufs-bundle ng-godas currently broken

Testing

Dependencies

n/a

travissluka and others added 4 commits February 28, 2024 09:30
* deprecate horizfilt/vertconv

* remove old tests

* remove 3dvar_godas

* remove SocaError

* remove horizfilt

* remove vertconv

* WIP

* add explicit_diffusion sqrt multiply

* fix more tests

* update tests

* increase tolerances
@travissluka
Copy link
Contributor

what was happening with the checkpoint test?

@climbfuji
Copy link
Contributor Author

what was happening with the checkpoint test?

It complained that the dependency for the test target couldn't be found

@travissluka
Copy link
Contributor

what was happening with the checkpoint test?

It complained that the dependency for the test target couldn't be found

hmm, oh well.
Changes seem reasonable

@climbfuji
Copy link
Contributor Author

Good news is that ufs-bundle CI passed, so I'll open this up for review

@climbfuji climbfuji marked this pull request as ready for review March 8, 2024 17:37
@climbfuji climbfuji requested a review from travissluka March 8, 2024 17:37
@travissluka travissluka merged commit 959ef23 into feature/ufs_dom Mar 8, 2024
@travissluka travissluka deleted the feature/ufs_dom_update_from_develop_20240308 branch March 8, 2024 19:51
@climbfuji
Copy link
Contributor Author

Thanks @travissluka !

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 this pull request may close these issues.

2 participants