-
Notifications
You must be signed in to change notification settings - Fork 85
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
Updated AMS models #1020
base: master
Are you sure you want to change the base?
Updated AMS models #1020
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, some clean up needed and some consistency across different AMS files needs to be added
da815d1
to
9d79763
Compare
On NVIDIA GPU:
|
7b31841
to
cbba122
Compare
Does anyone know why the CI fails here? |
@marchdf the docker container we use to drive CI is busted. I put in a fix for this to |
@psakievich can we merge this? I am worried that a bunch of nalu-wind modernization is going to pass us by... |
@marchdf I would have merged it but the CI has been failing. I think it might be good to do a mini-hack-a-thon to update some of these PR's. I don't want this one to stay in purgatory any longer, but I also have quite a bit of changes to how fields are registered coming down the pipeline. I think it will make the code simpler. I am not planning to start that work until I get tooling updated a bit though. So I might suggest waiting until I get that started... |
Still left to do:
Co-authored-by: Jeremy Melvin [email protected]