Skip to content

Conversation

Tokazama
Copy link
Member

Part of the problem that #297 was working around was the absence of any information related to how an index type translates into the shape of the resulting array. Some of the problems discussed in #312 are also related to this. I'll be following this up with a couple internal changes in ArrayInterface, but I want to do this piecewise to ensure I don't break anything by changing it all at once.

@codecov
Copy link

codecov bot commented Jun 13, 2022

Codecov Report

Merging #314 (15ce4ba) into master (d9b5089) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #314   +/-   ##
=======================================
  Coverage   91.49%   91.49%           
=======================================
  Files           9        9           
  Lines        1399     1399           
=======================================
  Hits         1280     1280           
  Misses        119      119           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d9b5089...15ce4ba. Read the comment docs.

@Tokazama Tokazama requested a review from chriselrod June 13, 2022 16:37
@Tokazama Tokazama merged commit 1fb77f5 into master Jun 13, 2022
@Tokazama Tokazama deleted the index-dims branch June 13, 2022 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants