Skip to content

Comments

Experimental nodes table#339

Draft
AnniekStok wants to merge 18 commits intomainfrom
experimental_nodes_table
Draft

Experimental nodes table#339
AnniekStok wants to merge 18 commits intomainfrom
experimental_nodes_table

Conversation

@AnniekStok
Copy link
Collaborator

A first attempt at having an interactive table widget in addition to the tree view. Right now it is a simple table view of all nodes and their features, with mirrored click behavior for selecting, appending, and centering nodes.
Discussed in #325. The table order is currently not changed by the selection (it is actually tricky because it would 'jump' while you are trying to select additional nodes).

table_widget

@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2026

Codecov Report

❌ Patch coverage is 74.82014% with 70 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.37%. Comparing base (4bdc866) to head (8184610).

Files with missing lines Patch % Lines
.../data_views/views/tree_view/custom_table_widget.py 72.31% 67 Missing ⚠️
...er/data_views/views/tree_view/tree_widget_utils.py 86.36% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
- Coverage   86.08%   85.37%   -0.71%     
==========================================
  Files          52       53       +1     
  Lines        4313     4569     +256     
==========================================
+ Hits         3713     3901     +188     
- Misses        600      668      +68     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cmalinmayor
Copy link
Collaborator

As discussed:

  • love it! 🎉
  • need to add selection keybinds now that they are merged to main
  • next steps would likely be filtering, but we merge this first and make new PR

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.

3 participants