-
Notifications
You must be signed in to change notification settings - Fork 407
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
Implement fill_random for DynRankView #4763
Implement fill_random for DynRankView #4763
Conversation
std::to_string(rank(a))); | ||
} | ||
} | ||
|
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.
If I could make this variadic in the View
ish arguments, then it could be used for the non-scalar deep_copy
implementations, too. It would then have to handle mixes of View
and DynRankView
. With C++17, that could just be a folded logical-or over the condition in the enable_if
.
The putative use case for the |
I was wondering about perf impact, but the original implementation is SOOO shitty perf wise that it doesn't matter I guess .... |
Fixes #3437.