-
Notifications
You must be signed in to change notification settings - Fork 352
-
Notifications
You must be signed in to change notification settings - Fork 352
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
Tagcounters.json no longer being picked up for custom reporting? #286
Comments
Ah yes there is code to exclude "Metric" tags from the report which are deemed non-essential i.e. the purpose of the primary report on the HTML UI features page is...features not metrics. They will continue to appear in the Summary Tag Counters page. Any non-metric tags and associated rules including custom ones will appear in the features section as designed. I will update the wiki for additional clarity about this. Thanks for asking. |
Updated page to clarify. |
Guy I'm not sure this is working correctly. From the two screenshots we see that Tag Counters found, for example, 118 matches for Function Defined, but the first screenshot has no confidence, N/A and no light up for function defined. What does the indicator in the first screenshot mean if its not related to these numbers in the second one? |
Metrics are not supported in the Feature UI page hence the partial result. I suppose I can fix that and block them from appearing at all. I'll reopen as a bug but will not be displaying metrics on that page. |
Basically, each Metric or counter is excluded from the details match results only the tag is counted but because the tag was added to the UI group it partially appears with no data...just didn't block it from being added as a tag group and it should due to aforementioned reasons. Thanks I'll fix that aspect. |
Final comment to help explain that an early design decision not include include metrics in the detailed report is that they would appear as noise throughout the json report with each instance taking up space when we just want a count. Again, the primary purpose is to identify features. The exclusion of these from the detailed matches is what creates the empty data rows in the features page today because we didn't intend for them to appear here, only the tag counters area. Hope that helps. |
One other change that was made a few versions back is we no longer need the tagcounters.json to define the behavior to automatically count anything with the tag section containing "metric" and not include it in the detailed reports. Looks like my documentation didn't get updated which is part of the reason for the confusion so I've corrected that in the tags section of the wiki and in the custom reports section. |
Thanks for taking the time to reply, and for updating the docs. In my opinion it's a shame that the option to include metrics in the detailed report was removed, as we'd find this information useful in our use case. |
The information is still available just not on the "features" page which represents functionality only and is tied to the detailed matches in the report. I do understand it's just in this case there is a design/intent conflict but hope you enjoy the tool and can make use of the counters info found on the summary subpage of the html report and in the json/text formats. |
Detection and warning added to log. Final doc updated. |
Description
Following the documentation here, I wanted to see detail of Classes and Functions in the reports, so I created this tagcounters.json file in the preferences directory:
and then added this to tagreportgroups.json:
Running an analysis results in the custom Code Metrics section being added to the report, but no results for Class Defined or Function Defined are shown.
Furthermore on the Summary - Tag Counters screen, it displays:
So it appears to be picking up the customisations in tagreportgroups.json, but not in tagcounters.json.
Screenshots
Desktop
The text was updated successfully, but these errors were encountered: