Skip to content

Add tree_view for viewing Profile.print#249

Open
IanButterworth wants to merge 2 commits into
masterfrom
ib/tree_view
Open

Add tree_view for viewing Profile.print#249
IanButterworth wants to merge 2 commits into
masterfrom
ib/tree_view

Conversation

@IanButterworth

@IanButterworth IanButterworth commented Jun 2, 2025

Copy link
Copy Markdown
Collaborator

Reads in the output of Profile.print() and turns it into an expandable treeview, with cursor controls.
click expander icon or spacebar on selected row to expand/collapse, click row or return key on selected row to edit.

A probably better and less brittle implementation would be to use the same read mechanism as ProfileView.view, but this was quite easy (and fun) to build and it maintains the styling of Profile.print which is nicely consistent.

julia> ProfileView.view_tree()
Screen.Recording.2025-05-30.at.5.17.59.PM.mov

@timholy

timholy commented Jun 25, 2026

Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jun 25, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Resolved. The merge conflicts in Project.toml have been fixed — keeping version 1.10.3 and updating the Cthulhu compat to "2, 3" from master. All other files merged cleanly. Merge commit: 4171d8b.

Copilot AI requested a review from timholy June 25, 2026 11:08

@timholy timholy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Seems OK to me; it's legit to parse the printed output, though the alternative would also have been legit.

It does lack tests, but perhaps that's OK for a GUI package. Merge at will.

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