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
Add support for ComplexDataVector to Pypp tests #1398
Conversation
fe1c346
to
c4f9c46
Compare
c4f9c46
to
76b5fe5
Compare
squashed in iwyu and clang-tidy changes to make Travis happy. |
@@ -143,7 +143,7 @@ void check_with_random_values_impl( | |||
std::index_sequence<ResultIs...> /*index_return_types*/, | |||
std::index_sequence<ArgumentIs...> /*index_argument_types*/, | |||
std::index_sequence<MemberArgsIs...> /*index_member_args*/, | |||
TagsList /*meta*/, const double epsilon = 1.0e-12) { | |||
TagsList /*meta*/, double epsilon = 1.0e-12) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is fixed more correctly in #1389 so my vote is let's ignore it :) clang-tidy is wrong here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, sounds good. Removed.
76b5fe5
to
0008d67
Compare
0008d67
to
cee1036
Compare
clang-tidy failure is false positive address in #1389 |
Proposed changes
Python (and numpy) routines gracefully handle complex values, so it's just a matter of bringing the values from the
ComplexDataVector
over to the numpy structures.Types of changes:
Component:
Code review checklist
clang-tidy
andIWYU
.For instructions on how to perform the CI checks locally refer to the Dev
guide on the Travis CI.
make doc
to generate the documentation locally intoBUILD_DIR/docs/html
.Then open
index.html
.code review guide.