-
Notifications
You must be signed in to change notification settings - Fork 0
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
実行スペースを指定できるようにする #200
base: support-more-precisions
Are you sure you want to change the base?
実行スペースを指定できるようにする #200
Conversation
GPUでのテスト修正
add circuit batch update
include/scaluq/types.hpp
Outdated
template <Precision Prec> | ||
using Matrix = Kokkos::View<Complex<Prec>**, Kokkos::LayoutRight>; | ||
template <Precision Prec, ExecutionSpace Space> | ||
using Matrix = Kokkos::View<Complex<Prec>**, Space>; |
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.
LayoutRightやめてパフォーマンス変わりませんか
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.
#200 (comment) と同じ理由で消えてました!
戻します
{(std::int64_t)(state.dim() >> std::popcount(target_mask | control_mask)), | ||
(std::int64_t)values.size()}), |
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.
static_castではだめだったんですか?
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.
手元でテストする際にいろいろ試していたら変わってたみたいです!
とくに理由はないので差し戻します.
close #140
support-more-precision ブランチにmergeしたいです.テストは編集していません