-
Notifications
You must be signed in to change notification settings - Fork 13
cs/Implementing the utility network load report #408
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a utility network load report sample that allows users to create electric distribution reports showing customer counts and total load per phase by tracing downstream from a given point in the Naperville Electric utility network.
Key Changes:
- Added interactive UI with phase selection and report generation capabilities
- Implemented downstream trace functionality with custom trace configuration for service points and load calculations
- Created comprehensive documentation explaining the load report use case and implementation
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| lib/samples/create_load_report/create_load_report.dart | Main implementation of the load report sample with utility network tracing, phase selection UI, and report generation logic |
| lib/samples/create_load_report/README.metadata.json | Metadata configuration defining the sample's category, keywords, and relevant APIs |
| lib/samples/create_load_report/README.md | User documentation explaining the use case, usage instructions, and implementation details |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| else | ||
| LoadingIndicator( | ||
| visible: !_ready, | ||
| text: 'Initializing utility network...', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The loading indicator should be on top of the Column widget, not replace it when the _phaseDataList is empty. It might be better to introduce another widget that is displayed when _phaseDataList is empty, while keeping LoadingIndicator for when UN has not initialized yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
No description provided.