sandialabs / Albany

Sandia National Laboratories' Albany multiphysics code
Other
282 stars 89 forks source link

A bunch of land ice failing tests in Albany #1077

Closed ikalash closed 1 month ago

ikalash commented 2 months ago

A bunch of tests are failing:

https://sems-cdash-son.sandia.gov/cdash/viewTest.php?onlyfailed&buildid=218512

In looking at the errors, it seems at least some of them are due to changes in Albany (Cubature Degrees (Horiz Vert) an unknown parameter), but some may be due to changes to Trilinos. Could someone please have a look at these?

bartgol commented 2 months ago

Still working on this.

ikalash commented 2 months ago

A bunch more stuff got broken today in terms of the builds. You probably know already @bartgol but letting you know just in case.

bartgol commented 2 months ago

This kind of build errors is unrelated to my recent changes. Perhaps STK finally removed deprecated code for fields?

https://sems-cdash-son.sandia.gov/cdash/viewBuildError.php?buildid=227393

ikalash commented 2 months ago

@alanw0 : we are getting STK-related errors in Albany:

https://sems-cdash-son.sandia.gov/cdash/viewBuildError.php?buildid=227393

 struct FieldRank<stk::mesh::Field<Scalar,Tag1,Tag2,Tag3,Tag4,Tag5,Tag6,Tag7>> {

Have STK removed some more deprecated code?

alanw0 commented 2 months ago

Sorry about that, let me bring @djglaze into this. I think the belief was that this (final?) removal of deprecated field code should have no impact since it had already been deprecated... Dave will know more details, and can suggest the best action here...

djglaze commented 2 months ago

You're right that this is due to another round of deprecated code removal. The timeline is a bit complex, so here's a quick summary:

I think you're caught in the 9/2024 changes, which included things like removing the extra template parameters from the stk::mesh::Field type that became completely inaccessible with the 7/2024 changes. You should be able to change:

struct FieldRank<stk::mesh::Field<Scalar,Tag1,Tag2,Tag3,Tag4,Tag5,Tag6,Tag7>> {

into

struct FieldRank<stk::mesh::Field<Scalar>> {

to get building again. The extra tags were no longer used and no longer user-settable, so they were removed. Sorry about all the churn. This rollout strategy was the least-painful way we could think of to incrementally make this kind of change to STK mesh.

bartgol commented 2 months ago

I fixed the field error in Albany, but I'm getting linking errors due to this STK problem. I fixed it locally in my repo, so that I can verify tests run correctly.

bartgol commented 1 month ago

Commit 5d400c176 fixed all tests currently failing on my workstation. We can keep the issue open, and close it on Monday if we see tests passing on cdash.

Thanks for your patience while I track these down!