Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace the IndexSet in the deephaven.learn package with a RowSet #1636

Closed
chipkent opened this issue Dec 4, 2021 · 0 comments · Fixed by #1637
Closed

Replace the IndexSet in the deephaven.learn package with a RowSet #1636

chipkent opened this issue Dec 4, 2021 · 0 comments · Fixed by #1637
Assignees
@chipkent
Labels
DocumentationNeeded feature request New feature or request triage
Milestone
Nov 2021

Comments

@chipkent
Copy link
Member

chipkent commented Dec 4, 2021

The IndexSet in the deephaven.learn package can be replaced with a RowSet to yield a more efficient implementation by allowing the chunk interface to fill arrays.
@chipkent chipkent added this to the Nov 2021 milestone Dec 4, 2021
@chipkent chipkent self-assigned this Dec 4, 2021
@chipkent chipkent mentioned this issue Dec 4, 2021
Merged
mofojed added a commit that referenced this issue Dec 22, 2023
@github-actions @deephaven-internal @mofojed
Update web version 0.58.0 (#4982)
c245cc7 
Release notes https://github.com/deephaven/web-client-ui/releases/tag/v0.58.0

### Bug Fixes

* `figure_title` and `chart_title` were not mapped up correctly ([#1676](deephaven/web-client-ui#1676)) ([73e0b65](deephaven/web-client-ui@73e0b65)), closes [#1674](deephaven/web-client-ui#1674) [#1675](deephaven/web-client-ui#1675)


### Features

* "Group" column for rollup/tree tables ([#1636](deephaven/web-client-ui#1636)) ([ba1d51b](deephaven/web-client-ui@ba1d51b)), closes [#1555](deephaven/web-client-ui#1555)
* Add alt+click shortcut to copy cell and column headers ([#1694](deephaven/web-client-ui#1694)) ([4a8a81a](deephaven/web-client-ui@4a8a81a)), closes [deephaven/web-client-ui#1585](deephaven/web-client-ui#1585)
* Theming - Spectrum variable mapping and light theme ([#1680](deephaven/web-client-ui#1680)) ([2278697](deephaven/web-client-ui@2278697)), closes [#1669](deephaven/web-client-ui#1669) [#1539](deephaven/web-client-ui#1539)




---------

Co-authored-by: deephaven-internal <deephaven-internal@users.noreply.github.com>
Co-authored-by: mikebender <mikebender@deephaven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chipkent chipkent

Labels
DocumentationNeeded feature request New feature or request triage
Projects
None yet
Milestone
Nov 2021
Development

Successfully merging a pull request may close this issue.

Replace the IndexSet in the deephaven.learn package with a RowSet chipkent/deephaven-core
1 participant
@chipkent