Antti Viita
Antti Viita
Still having issues in getting merge results and allowing to run ReportGenerator from that. Following is the `dotnet test` command I am trying to run. Using coverlet 2.9.0 in my...
And I have tried different variations of this CoverletOutputFormat string, all resulting to error of some sort with command above: ``` /p:CoverletOutputFormat=\"json%2copencover\" /p:CoverletOutputFormat="json%2copencover" /p:CoverletOutputFormat='json,opencover' /p:CoverletOutputFormat='json%2copencover' ```
And found my problem, so I am not supposed to put the json file, but only a path to `CoverletOutput` param. Defining json file likely caused coverlet xml to be...
@pieterderycke Any chance of getting this merged?
I also ran the Jace benchmarks as those seemed to allow testing performance with and without case sensitivity. The tests are neclecting to test the basic use case though that...
Here are also the full excel files before and after the fix. Note that only the benchmarks with formula "something2 - (var1 + var2 * 3)/(2+3)" are affected when case...
@pieterderycke Created a PR #76 to fix the issue.
This bug can also cause some extra memory traffic. While this is likely Gen 0 Heap, it surely would not hurt to get it removed too. The fix should apply...
Thread safety would also make the cache usage more efficient. In my current implementation I am not sharing the CalculationEngine at all between objects. I could definitely do something about...
I could improve my code by creating the Engine per user, so I would be ensured to have no concurrency issues, but even that approach leaves two problems. 1. There...