Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat: Add the ability to export the Entity System trend data to a csv#397

Merged
codyward-skybox merged 3 commits into
mainMojang/minecraft-debugger:mainfrom
cody.ward/ecs_exportMojang/minecraft-debugger:cody.ward/ecs_exportCopy head branch name to clipboard
Jul 7, 2026
Merged

feat: Add the ability to export the Entity System trend data to a csv#397
codyward-skybox merged 3 commits into
mainMojang/minecraft-debugger:mainfrom
cody.ward/ecs_exportMojang/minecraft-debugger:cody.ward/ecs_exportCopy head branch name to clipboard

Conversation

@codyward-skybox

Copy link
Copy Markdown
Contributor

This change adds the ability to export the Entity and System trend data to a CSV for external analysis.

As part of this change, I've also ensured that using the Clear button will reset all the trend data. Additionally I've added the ability to increase the trend data capture duration via user dropdown, similar to how we handle the CPU Profiler capture window.

Comment thread src/panels/minecraft-diagnostics.ts Outdated
saveLabel: 'Export',
defaultUri: workspaceFolderUri ? Uri.joinPath(workspaceFolderUri, suggestedFileName) : undefined,
filters: {
'CSV Files': ['csv'],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like there's a warning here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the VS Code API fighting with our linter, these need to be human readable: https://code.visualstudio.com/api/references/vscode-api#SaveDialogOptions

I'm thinking I might make a follow up story to do a pass on our linter warnings and see if we can adjust some configurations to make everything happy, as this has come up numerous times now.

@codyward-skybox
codyward-skybox merged commit 5191375 into main Jul 7, 2026
2 checks passed
@codyward-skybox
codyward-skybox deleted the cody.ward/ecs_export branch July 7, 2026 17:29
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.