Skip to content

Fit the profile summary in a terminal - #85

Merged
bbatsov merged 2 commits into
mainfrom
compact-profile-table
Nov 6, 2026
Merged

bbatsov merged 2 commits into
mainfrom
compact-profile-table

Conversation

@bbatsov

@bbatsov bbatsov commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

,profile printed cider-nrepl's inspector table as it came, about 130 columns, with a row number and a samples column that only make sense in CIDER's inspector. It's now laid out like neorepl's other tables. The edn and JSON :summary stays text, in the new layout.

@bbatsov
bbatsov force-pushed the compact-profile-table branch 2 times, most recently from 0d93193 to 8606963 Compare November 6, 2026 13:44
cider-nrepl shows the summary in orchard's table view, which CIDER's
inspector can go into: a column with each row's number, one with a few
of the samples, :name and the like for headings and the vars as
neorepl now takes the cells from the view and lays them out as its other
tables are, without the number and the samples, with headings and names
written as they are elsewhere. Every cider-nrepl since 0.55, the first
with these ops, shows the summary that way.
neorepl profile --format edn or json had the summary as the text table,
which a script or an agent has to split back into columns.
features/profile-summary now takes the rows out of cider-nrepl's table,
a map for each function, with :n as a number and the times as orchard
writes them, like "4 us", and render only lays them out. orchard cuts a
cell at 70 characters, a long var's name too, which the data can't
undo.
@bbatsov
bbatsov force-pushed the compact-profile-table branch from 8606963 to 409a6d6 Compare November 6, 2026 18:40
@bbatsov
bbatsov merged commit 8c726f8 into main Nov 6, 2026
20 checks passed
@bbatsov
bbatsov deleted the compact-profile-table branch November 6, 2026 18:43
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.

1 participant