Add AQ Knowledge Distribution placeholders; add Division to treemap#104
Add AQ Knowledge Distribution placeholders; add Division to treemap#104ricardovogel wants to merge 6 commits intomainfrom
Conversation
Sigrid maintainability feedback✅ You wrote maintainable code and achieved your objective of 3.0 stars. Show detailsSigrid compared your code against the baseline of 2026-03-18. 👍 What went well?
👎 What could be better?
📚 Remaining technical debt
View this system in Sigrid to explore your technical debt ⭐️ Sigrid ratings
💬 Did you find this feedback helpful?We would like to know your thoughts to make Sigrid better. Sigrid Open Source Health feedback✅ You achieved your objective of having no open source vulnerabilities. ✅ You achieved your objective of having no open source libraries with license issues. Show detailsSigrid compared your code against the baseline of 2026-03-18. |
|
| | `PORTFOLIO_KNOWL_ABOVE_MARKET` | Percentage of systems scoring above market average (≥3.5 stars) on knowledge distribution quality. | | | ||
| | `PORTFOLIO_KNOWL_AVG_RATING` | Volume-weighted average knowledge distribution rating across all systems in the portfolio. | | | ||
| | `PORTFOLIO_KNOWL_BELOW_MARKET` | Percentage of systems scoring below market average (<2.5 stars) on knowledge distribution quality. | | | ||
| | `PORTFOLIO_KNOWL_MARKET_AVERAGE` | Percentage of systems scoring market average (2.5-3.5 stars) on knowledge distribution quality. | | |
There was a problem hiding this comment.
@maximjanssen
Did you implement average rating for the other AQ metrics too, or only the treemap?
Either way, I think the docs here are incorrect
There was a problem hiding this comment.
@ricardovogel I was halfway through making these changes when Floris and I decided to put this MR on hold until the v1 changes are done. That's why everything is a bit out of sync now.



Added placeholders for knowledge distribution specifically.
Additionally: added "division" as a possible grouped_by placeholder in the treemap