Skip to content

Conversation

@alexander-yakushev
Copy link
Member

@alexander-yakushev alexander-yakushev commented Apr 25, 2025

First part is pretty minor – allow analytics for all maps and for arrays.

The second part is a bit more involved. Sometimes, it is beneficial to display a map as map after you order it somehow – sort by keys or values or anyhow else. However, turning that sorted list of MapEntries back into an array-map just so that inspector or printer displays it properly without messing up the entry order is inefficient. Thus, I've added a meta flag that is interpreted by inspector, printer, and pretty-printer as "show this sequence of mapentries as a map".

This is currently only used for displaying frequencies computed by inspector analytics. We have tests for that functionality, so no new tests are necessary.


  • You've updated the changelog (if adding/changing user-visible functionality)

@alexander-yakushev alexander-yakushev force-pushed the inspector branch 2 times, most recently from 94fa994 to 8090d70 Compare April 25, 2025 20:47
@alexander-yakushev
Copy link
Member Author

Update: I came up with a much simpler solution for producing maps sorted by values.

@alexander-yakushev alexander-yakushev merged commit bb6fe7c into master Apr 26, 2025
20 checks passed
@alexander-yakushev alexander-yakushev deleted the inspector branch April 26, 2025 20:03
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.

2 participants