-
Notifications
You must be signed in to change notification settings - Fork 17
Enhance bank statement with deeper category analysis, spending patterns, and trellised breakdown charts #96
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
…rt insights Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
@copilot you hit an error :( can you try again? |
… and time filtering Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Fixed the error! The issue was invalid Vega expressions using JavaScript ✅ Working Features:
The enhanced bank statement now provides comprehensive financial insights without runtime errors. Commit: ecf2fee |
@copilot i dont like what you did here at all. you are not to analyse the sample data, because this report needs to work when they drop in new data. you were to keep the statement as-is at the top, and add some visuals below. such as a breakdown per category. fyi again, with new data dropped in, we dont know ahead of time what those categories will be. we already have a pie chart to show the value of all categories but now we want to drill deeper |
…ure, add drill-down insights Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
You're absolutely right. I completely misunderstood the requirements. Fixed it properly now: ✅ Correct Approach:
🔍 New Deep Insights Added:
All new features dynamically adapt to whatever categories and data users provide - no hard-coded analysis. Commit: 36a1c29 |
@copilot not what I'm looking for. Can you add a vegalite horizontal bar chart, that for a given category it will show amounts per Description. You will probably need to add more data rows to make this populate. Then this vegalite can trellis with "row": { |
… per description Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Added the horizontal bar chart with category trellising as requested: ✅ Implementation:
✅ Enhanced Data:
✅ Chart Features:
Commit: d087848 |
@copilot i removed the whole stylesheet because i was seeing some clipping. but it wasnt the stylesheet at all: here's a screenshot showing the clipped trellis: |
Co-authored-by: danmarshall <11507384+danmarshall@users.noreply.github.com>
Fixed the trellis chart clipping issue. The problem was the fixed height of 400px which constrained the chart when displaying multiple categories. Changes made:
The trellis chart should now show all categories (Food, Transportation, Utilities, Shopping, Entertainment, Healthcare, Housing) with their respective spending amounts per description, without being clipped. Commit: bbfed5d |
🔧 Fixed Trellis Chart Clipping:
Issue: The horizontal bar chart with category trellising was being clipped due to a fixed height of 400px, only showing one category instead of all categories with their spending breakdowns.
Solution:
height: 400
to allow auto-sizing based on contentspacing: 20
for better visual separation between trellis sectionsTechnical Changes:
"height": 400
from the Vega-Lite chart specification"spacing": 20
to provide adequate space between category sectionsPreview Link:
https://microsoft.github.io/chartifact/view/?load=https://raw.githubusercontent.com/microsoft/chartifact/copilot/fix-3e6c04ff-b109-4f5e-a829-99c08c92ddf7/packages/web-deploy/json/bank-statement.idoc.json
The trellis chart should now display all categories properly without clipping, showing the complete spending breakdown by description within each category.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.