Skip to content
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

STK: install error #6522

Closed
ikalash opened this issue Jan 4, 2020 · 14 comments
Closed

STK: install error #6522

ikalash opened this issue Jan 4, 2020 · 14 comments
Labels
client: Albany Issue impacting the Albany project pkg: STK type: bug The primary issue is a bug in Trilinos code or tests

Comments

@ikalash
Copy link
Contributor

ikalash commented Jan 4, 2020

It looks like there is a Trilinos installation error due to STK showing up in the Albany nightlies:

https://my.cdash.org/viewBuildError.php?buildid=1754035
http://cdash.sandia.gov/CDash-2-3-0/viewBuildError.php?buildid=92142

Here is the error:

CMake Error at packages/stk/stk_util/stk_util/cmake_install.cmake:64 (file):
  file INSTALL cannot find
  "/.../build/TriBuild/packages/stk/stk_util/stk_util/STK_Trilinos_config.h".
Call Stack (most recent call first):
  packages/stk/stk_util/cmake_install.cmake:69 (include)
  packages/stk/cmake_install.cmake:50 (include)
  cmake_install.cmake:110 (include)

Could you please have a look at / fix this?

@trilinos/stk

@ikalash ikalash added type: bug The primary issue is a bug in Trilinos code or tests pkg: STK client: Albany Issue impacting the Albany project labels Jan 4, 2020
@alanw0
Copy link
Contributor

alanw0 commented Jan 4, 2020

Hi @ikalash, sorry about that! This is the same issue referenced in #6521, and I will have it either fixed or reverted before the end of tomorrow.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 4, 2020

Thanks, Alan! Sorry about creating the duplicate issue - I must have missed #6521.

@alanw0
Copy link
Contributor

alanw0 commented Jan 5, 2020

A fix has been pushed to the develop branch for this. Let me know if you still see any issues.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 6, 2020

@alanw0 : I just checked our dashboard and while Trilinos builds/installs correctly now, it appears we have compilation errors in Albany having to do with STK:

/.../repos/Albany/src/disc/stk/Albany_AsciiSTKMesh2D.cpp(245): error: namespace "stk::mesh" has no member "set_cell_topology"

http://cdash.sandia.gov/CDash-2-3-0/viewBuildError.php?buildid=92176

Did the stk::mesh::set_cell_topology routine in STK go away / change?

There's a similar error with get_cell_topology:

 /.../repos/Albany/src/disc/stk/Albany_AsciiSTKMesh2D.cpp(255): error: class "stk::mesh::MetaData" has no member "get_cell_topology"

@alanw0
Copy link
Contributor

alanw0 commented Jan 6, 2020

@ikalash, yes some methods were deprecated. Our procedure is to decorate functions/methods with the STK_DEPRECATED macro, then after some time (at least 6 weeks) delete them. In this case it was more like 6 months. I've complained to the trilinos-frameowrk folks that the SHOW_DEPRECATED_WARNINGS flag should be on by default, because nobody ever sees our deprecations until we delete the method in question and then it becomes an error instead of a warning.
Anyway, in this case, the MetaData class only deals in stk::topology now. But in the MetaData.hpp header there are functions get_topology and get_cell_topology that can translate back and forth between stk::topology and shards::CellTopology.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 6, 2020

Seems this affects SCOREC as well:

/home/ikalash/nightlyAlbanyTests/Results/Trilinos/SCOREC/stk/apfMeshSTK.cc:209:14: error: ‘set_cell_topology’ is not a member of ‘stk::mesh’; did you mean ‘get_cell_topology’?
  209 |   stk::mesh::set_cell_topology(part, topo);
      |              ^~~~~~~~~~~~~~~~~
      |              get_cell_topology
/home/ikalash/nightlyAlbanyTests/Results/Trilinos/SCOREC/stk/apfMeshSTK.cc: In function ‘void apf::copyMeshToMeta(apf::Mesh*, apf::StkModels&, apf::StkMetaData*)’:
/home/ikalash/nightlyAlbanyTests/Results/Trilinos/SCOREC/stk/apfMeshSTK.cc:227:11: error: ‘apf::StkMetaData’ {aka ‘class stk::mesh::MetaData’} has no member named ‘register_cell_topology’; did you mean ‘register_topology’?
  227 |     meta->register_cell_topology(topo[i], ranks[i]);

I will open an issue with the SCOREC folks.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 7, 2020

@alanw0 : I'm working on converting Albany to work with the new STK routines, and I had a quick question - is there an analog of get_cell_topology_root_part for the class stk::mesh::MetaData?

@ikalash
Copy link
Contributor Author

ikalash commented Jan 7, 2020

Never mind - silly me, it's get_topology_root_part.

@alanw0
Copy link
Contributor

alanw0 commented Jan 7, 2020

Sorry I somehow missed your question earlier. But yes you are right, get_topology_root_part.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 7, 2020

No problem - I think webmail was malfunctioning... I got ~30 notifications this afternoon that were from the morning.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 7, 2020

I'm almost done fixing up Albany to get it to compile with STK VOTD. I've had to switch from shards traits to stk::topology for the various elements. I was curious, what is the distinction b/w elements such as LINE_2 and LINE_2_1D and QUAD_4 and QUAD_4_2D? I noticed in modifying the code that it matters which of these is used.

@alanw0
Copy link
Contributor

alanw0 commented Jan 7, 2020

Yes it's kind of a confusing distinction. QUAD_4 is a face, and can only be defined in a 3D mesh. QUAD_4_2D can only be defined in a 2D mesh, but then it's an element and not a face. To be more general, QUAD_4 is a side in a 3D mesh and LINE_2 is a side in a 2D mesh. LINE_2_1D is an element in a 1D mesh.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 8, 2020

Ah ok, that explains some of the errors I was still seeing for certain tests. The distinction is critical in implementing a correct fix - thanks for the clarification.

@ikalash
Copy link
Contributor Author

ikalash commented Jan 8, 2020

I've fixed Albany to compile/run with STK VOTD. Thanks again for your help @alanw0 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client: Albany Issue impacting the Albany project pkg: STK type: bug The primary issue is a bug in Trilinos code or tests
Projects
None yet
Development

No branches or pull requests

2 participants