-
Notifications
You must be signed in to change notification settings - Fork 152
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
(chore): add Index
type to __getitem__
for sparse dataset
#1648
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1648 +/- ##
==========================================
- Coverage 86.87% 84.41% -2.47%
==========================================
Files 39 39
Lines 6033 6036 +3
==========================================
- Hits 5241 5095 -146
- Misses 792 941 +149
|
With “doc rendering fix”, you mean making it so “TypeVar” gets rendered more nicely or … ? |
YEs you had mentioned wanting it, and it maybe being a blocker? But I would merge as-is as well....so gonna request your review then...and if you want to block on it, ok |
Co-authored-by: Philipp A. <flying-sheep@web.de>
Starting point. I think this should go in the next 0.11 release candidate (maybe along with your doc rendering fix @flying-sheep but doesn't have to). I would also be in favor of exporting this type since it is used elsewhere in quite important places.