Skip to content

docs: DOC-615: deephaven design v2#7715

Open
margaretkennedy wants to merge 14 commits intodeephaven:mainfrom
margaretkennedy:doc-615-deephaven-design-v2
Open

docs: DOC-615: deephaven design v2#7715
margaretkennedy wants to merge 14 commits intodeephaven:mainfrom
margaretkennedy:doc-615-deephaven-design-v2

Conversation

@margaretkennedy
Copy link
Contributor

Forking workflow

@github-actions
Copy link
Contributor

github-actions bot commented Feb 24, 2026

Deploying docs previews for 9cef1ca (available for 14 days)

Python
Groovy

@margaretkennedy margaretkennedy added documentation Improvements or additions to documentation NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed. labels Feb 24, 2026
Copy link
Member

@mofojed mofojed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're going into way too much architectural detail IMO for this overview page. Also a bunch of it is incorrect or irrelevant, I'd much rather just link to the deephaven.ui documentation (which we'll also keep up to date if things change): https://deephaven.io/core/ui/docs/#key-features
Or at the very least, copy the information from there, rather than hallucinating stuff that isn't accurate.
Also, under "groovy" we really should just link to Python and ui.resolve, or the web-client-ui. Having the rest of the detail there is another place where docs can rot.

Copy link
Contributor

@elijahpetty elijahpetty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments should also be applied to Python if they are accepted.

- **Wide range of components**: From simple text fields to complex tables and plots

#### Displaying a grid
For complete documentation, tutorials, and examples, see the [`deephaven.ui` documentation](https://deephaven.io/core/ui/docs/).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this indent intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

margaretkennedy and others added 6 commits February 25, 2026 14:16
Co-authored-by: Larry Booker <lbooker42@gmail.com>
Co-authored-by: elijahpetty <128415452+elijahpetty@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants