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

enable filtering on overview is not applicable #116

Closed
Lorilatschki opened this issue Nov 28, 2017 · 10 comments
Closed

enable filtering on overview is not applicable #116

Lorilatschki opened this issue Nov 28, 2017 · 10 comments

Comments

@Lorilatschki
Copy link

Hi,

we have an issue with our generated overview of our opencover results. We have covered about 15 assemblies so we want to collapse all of them in the overview. But this always hangs or is extremly slow. Also collapsing one assembly hangs. Seems to be an issue with the underlaying java command or the amount of data but we have no glue how to solve this.

Does anybody knowing this problem?

@danielpalme
Copy link
Owner

danielpalme commented Nov 28, 2017

Could you please send me your index.html and the corresponding combined.js file?

@Lorilatschki
Copy link
Author

Lorilatschki commented Nov 28, 2017

This email address is not correct. I always get i mailer deamon.

@danielpalme
Copy link
Owner

The address is correct.

@Lorilatschki
Copy link
Author

I have uploaded the data on a share and you should got a mail with the link.

@danielpalme
Copy link
Owner

Just received your sample report. Will have a look at it and see if performance can be improved.

@Lorilatschki
Copy link
Author

Any news from your side?

@danielpalme
Copy link
Owner

I can confirm, that is the page is pretty slow. The problem is not the filtering itself.
The rendering of the table takes most of the time.

I have to check, if there is a way to speed things up. But this will not happen within the next days.
At the moment I'm too busy to take care of that.

@danielpalme
Copy link
Owner

danielpalme commented Dec 28, 2018

I finally had time to improve the performance on the summary page.
I had to rewrite the complete report (now using Angular 7 instead of ReactJS). Now it seems to perform much better.

Here's the new release:
https://www.nuget.org/packages/ReportGenerator/4.0.5
https://www.nuget.org/packages/dotnet-reportgenerator-cli/4.0.5
https://www.nuget.org/packages/dotnet-reportgenerator-globaltool/4.0.5

@Lorilatschki
Copy link
Author

Thanks for the update.
I will try it out when I am back in office (07.02). Keep you informed.

@Lorilatschki
Copy link
Author

I have tried the new package 4.0.11 and it turned out that the performance for Collapse all | Expand all on the summary page is much better than before.
You can close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants