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

Collect Kelvin vector code and function in single place. #2060

Merged
merged 5 commits into from Feb 5, 2018

Conversation

Projects
None yet
1 participant
@endJunction
Copy link
Member

endJunction commented Jan 29, 2018

Commit-wise will be faster....

There are still many places having component-wise sigma/epsilon output, I'm aware of this, but it will take another five PRs to replace all of them. (Help is welcome;)

@endJunction endJunction force-pushed the endJunction:ExtendKelvinVectorFunctions branch 3 times, most recently from aec9558 to 9f8f6bd Jan 30, 2018

@endJunction

This comment has been minimized.

Copy link
Member Author

endJunction commented Jan 30, 2018

The failing build is due to a different Doxygen problem, not related to these changes.

@endJunction endJunction force-pushed the endJunction:ExtendKelvinVectorFunctions branch from 9f8f6bd to eb41efd Jan 31, 2018

endJunction added some commits Jan 29, 2018

[MaL] Rename kelvinVectorToTensor().
Also specify all of the template arguments as needed for the MSVC
compiler.
[MaL] Add KelvinVector to symmetric tensor cnv.
This is giving a shortened version of the matrix
in a vector form.

@endJunction endJunction force-pushed the endJunction:ExtendKelvinVectorFunctions branch from eb41efd to c3b2b1e Feb 5, 2018

@endJunction endJunction merged commit 6bb45d1 into ufz:master Feb 5, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details

@endJunction endJunction deleted the endJunction:ExtendKelvinVectorFunctions branch Feb 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.