You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The problem with the periodic case is that the ddc::coordinate(idx_range.back()) do not return the maximum value on the domain.
Eg. on \([0, 2\pi]\) , it returns \(2\pi - d\theta\).
So the way the operator is written is not adapted to this case.
One idea is that for the periodic case, we can work with idx_range.back() +1 instead of idx_range.back() and ddc::coordinate(target_idx_max) + ddc::distance_right(target_idx_max) for the last point. At the end, correct it.
The text was updated successfully, but these errors were encountered:
The problem with the periodic case is that the
ddc::coordinate(idx_range.back())
do not return the maximum value on the domain.Eg. on \([0, 2\pi]\) , it returns \(2\pi - d\theta\).
So the way the operator is written is not adapted to this case.
One idea is that for the periodic case, we can work with
idx_range.back() +1
instead ofidx_range.back()
andddc::coordinate(target_idx_max) + ddc::distance_right(target_idx_max)
for the last point. At the end, correct it.The text was updated successfully, but these errors were encountered: