Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@akva2
Copy link
Member

@akva2 akva2 commented Jul 16, 2025

These fundamental types will also be useful in the state classes in this repo so let's move them up the chain.

@akva2 akva2 added the manual:irrelevant This PR is a minor fix and should not appear in the manual label Jul 16, 2025
@akva2
Copy link
Member Author

akva2 commented Jul 16, 2025

jenkins build this opm-simulators=6373 please

@akva2 akva2 force-pushed the import_voigt_symm branch from 5cdd392 to bdafd71 Compare July 16, 2025 09:45
@akva2
Copy link
Member Author

akva2 commented Jul 16, 2025

jenkins build this opm-simulators=6373 please

@akva2 akva2 force-pushed the import_voigt_symm branch from bdafd71 to c256260 Compare July 16, 2025 09:50
@akva2
Copy link
Member Author

akva2 commented Jul 16, 2025

jenkins build this opm-simulators=6373 please

Copy link
Member

@bska bska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much appreciated. I'll merge this and its downstream companion PR into the master branch.

@bska bska merged commit b585e64 into OPM:master Jul 16, 2025
2 checks passed
@akva2 akva2 deleted the import_voigt_symm branch July 16, 2025 10:17
@bska
Copy link
Member

bska commented Jul 16, 2025

For what it's worth, I just ran into a local build failure following this PR. Both VoigtArray.cpp and SymmTensor.hpp include <dune/common/fvector.hh>–to support interfaces taking Dune::FieldVector<double,N> (VoigtContainer constructor and SymmTensor::traction()).

Is this the point where we finally need to make OPM-Common depend on dune-common?

@akva2
Copy link
Member Author

akva2 commented Jul 16, 2025

sorry forgot about that, should be taken care of in #4662

no need to change policies, we'll just not build the code as we do for some other stuffs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants