Skip to content

Implement import/export in new version#20

Merged
FelixCAAuer merged 5 commits intomainfrom
feature/ImpExp
Oct 30, 2025
Merged

Implement import/export in new version#20
FelixCAAuer merged 5 commits intomainfrom
feature/ImpExp

Conversation

@FelixCAAuer
Copy link
Copy Markdown
Contributor

No description provided.

@FelixCAAuer FelixCAAuer requested a review from Copilot October 29, 2025 16:58
@FelixCAAuer FelixCAAuer self-assigned this Oct 29, 2025
@FelixCAAuer FelixCAAuer added the enhancement New feature or request label Oct 29, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 pull request adds support for a new Power_ImportExport table that replaces the previous Power_ImpExpHubs and Power_ImpExpProfiles tables, consolidating import/export data into a single file. It introduces a "grouped columns" feature to the Excel reader/writer framework to handle multiple hub-node pairs with repeating column structures.

  • Introduces Power_ImportExport table to replace Power_ImpExpHubs and Power_ImpExpProfiles
  • Implements "grouped columns" feature for handling repeating column patterns in Excel files
  • Updates case study loading and scaling logic to use the new consolidated table

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tests/test_ExcelReaderWriter.py Adds test case for Power_ImportExport read/write functionality
data/example/Power_ImportExport.xlsx Adds example Excel file for the new Power_ImportExport table
TableDefinitions.xml Defines Power_ImportExport table structure with grouped columns
TableDefinition.py Adds grouped column support to Column class
ExcelWriter.py Implements grouped column writing logic and adds write_Power_ImportExport method
ExcelReader.py Implements get_Power_ImportExport method to read the new table format
CaseStudy.py Updates case study to use Power_ImportExport instead of the two separate tables

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@FelixCAAuer FelixCAAuer merged commit 037eca9 into main Oct 30, 2025
1 check passed
@FelixCAAuer FelixCAAuer deleted the feature/ImpExp branch October 30, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants