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

Provide more instantiations for std::complex data types. #5590

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

bangerth
Copy link
Member

@bangerth bangerth commented Dec 6, 2017

This is necessary to run some more functions like VectorTools::project for complex-valued right hand sides. The changes herein are mostly uncontroversial and so I package them up into a patch of their own -- their just mechanically adding instantiations and then ensuring that things actually compile (and when they don't, to add the necessary template arguments and change data types).

The functions I'm instantiating here are mostly for mixed complex-real arithmetic. I need this in VectorTools::project and similar places because the mass matrix should remain real, but it's applied to a complex-valued right hand side.

Part of #2033.

@bangerth
Copy link
Member Author

bangerth commented Dec 6, 2017

/run-tests

@bangerth
Copy link
Member Author

bangerth commented Dec 6, 2017

Passes my testsuite, btw.

@bangerth
Copy link
Member Author

bangerth commented Dec 7, 2017

@kronbichler -- did you want me to do anything when you approved the patch? Or is it ok as it is?

@kronbichler
Copy link
Member

No, I think it is fine. I was just waiting to see if anyone else has a comment. Since that seems not to be the case, I will merge.

@kronbichler kronbichler merged commit d4cf4cf into dealii:master Dec 8, 2017
@bangerth bangerth deleted the more-complex branch December 8, 2017 15:52
luca-heltai pushed a commit to luca-heltai/dealii that referenced this pull request Dec 8, 2017
Provide more instantiations for std::complex data types.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants