-
Notifications
You must be signed in to change notification settings - Fork 84
ProfilerService for Langium #1850
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
Conversation
nice. will have a look. will also try to adapt it in our codebase.
|
56794c1
to
f981600
Compare
@cdietrich Below are some outputs taken from domain-model example:
@msujew Can you have a look at the design ? There should be a very small runtime overhead when the profiler is disabled: when parsing/linking/validating a document an initial check (at root level) is done to check if the profiler is enabled or not. |
Is there any activity on this? We have found ourselves in need of a similar feature. Hopefully no one minds, I brought the branch up to date here, and toggled profiling off (with the ability to turn on via module constructor call, as well as the original start/stop). Tests seem to pass, and the profiling info is very useful! |
I've been pretty busy lately and don't have much time to revisit this topic. You can take the lead on this topic if you'd like. |
Superseeded by #1993. |
Hi,
I started to implement a profiler for the parser. It is enabled by default when not in production mode.
Outputs are generated in the console after each parsing (maybe too verbose).
With DomainModel example it looks like:
@msujew and @cdietrich , can you give me some feedback about this functionality ?