Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add optional module for performance measurements #612

Draft
wants to merge 4 commits into
base: release/12.0
Choose a base branch
from

Conversation

tortmayr
Copy link
Contributor

@tortmayr tortmayr commented Sep 9, 2024

  • Introduce an optional performance module that add performance measurements to core GLSP concepts.
  • Performance is simply measured using console.time which also allows inspection of the tracked events in the dev tools trace viewer
  • The module is opt-in and disabled by default. To enable it the &performance=true query parameter has to be added to the url

@martin-fleck-at
Copy link
Contributor

@ivy-cst @ivy-lli FYI

@tortmayr
Copy link
Contributor Author

@ivy-cst @ivy-lli FYI: I have rebased this branch on the latest master

@tortmayr tortmayr changed the base branch from stale/master to release/12.0 November 26, 2024 08:01
- Introduce an optional performance module that add performance measurements to core GLSP concepts.
- Performance is simply measured using console.time which also allows inspection of the tracked events in the dev tools trace viewer
- The module is opt-in and disabled by default. To enable it the &performance=true query parameter has to be added to the url
Measure actions when they are actually handled instead of on dispatch
(avoids inconsistencies due to action queue blocking)
Add in depth logging for set-model-command and model update
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