ndarray icon indicating copy to clipboard operation
ndarray copied to clipboard

Change `NdProducer::Dim` of `axis_windows()` to `Ix1`

Open jonasBoss opened this issue 2 years ago • 2 comments

See #1304

This breaks the public API because the method axis_windows() now returns a new AxisWindows struct. And the dimension of the NdProducer::Dim will always be Ix1.

If the old behavior is needed the method windows() can be used.

I was thinking about adding a method AxisWindows::into_windows(self) -> Windows but I don't think that this will see much use. Let me know if i should implement it.

jonasBoss avatar Jul 13 '23 09:07 jonasBoss

Is there anything holding this back that I can do do address?

The CI errors don't seem to be related to this PR.

jonasBoss avatar Aug 18 '23 16:08 jonasBoss

Is there anything holding this back that I can do do address?

The CI errors don't seem to be related to this PR.

No. Sorry this is waiting on me finding time to look into it again.

adamreichold avatar Aug 18 '23 16:08 adamreichold

Rebased and updated with rustfmt. Maintainer pushed to do that.

Oh, and I squashed everything together. I had to do that, conflict resolution (with rebase) is a bit tedious otherwise.

bluss avatar Aug 01 '24 19:08 bluss

Thank you @jonasBoss and @adamreichold for the work on this one. It looks good to me, let's delay no further :slightly_smiling_face:, I'll be ready to push the merge button soon.

bluss avatar Aug 01 '24 19:08 bluss