Tobias Ortmayr

Results 90 comments of Tobias Ortmayr

# Performance Benchmarks A simple theia extension has been implemented to measure the RPC performance with the theia example applications. This extension implements a simple `RpcTestService` which can be used...

Correct we if I'm wrong but I don't think a CQ for msgpackR is required. I tried to follow the guide for [new dependencies](https://github.com/eclipse-theia/theia/wiki/Registering-CQs#case-new-or-updated-npm-production-dependency) but it seems to be oudated...

Thanks for the review @msujew . I have rebased the PR and addressed your feedback.

> It seems like this change actually breaks the electron build for me. I get the following error (in the frontend) when starting the electron app: > > ``` >...

@kosl90 The maxium call stack errors should be fixed with https://github.com/eclipse-theia/theia/pull/11636.

### Performance Testing This PR greatly improves the performance of FileSystem calls from within a plugin/VS Code extension. To test this with have used a simply [VS Code extension ](https://github.com/eclipse-theia/theia/files/8846109/json-rpc-test-0.0.1.zip)...

Thanks for testing this change @tsmaeder. >I've done a little testing, and I noticed that I get a ton of cancellation error stack traces in the console > >Details Monaco...

Some of this issues might be related to the behavior we have encountered in https://github.com/eclipse-theia/theia/issues/11249. Also the potential improvements discussed in https://github.com/eclipse-theia/theia/issues/11159. I will work on these two issues and...

Sure I can resolve the conflicts and retest it under Windows.

@planger Is this something that we should fix for the 1.0.0 release? Or are you fine with the fact that the `MovmentRestrictor` API can currently no be used in combination...