Skip to content
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

Feature/dgc 2055 get batch size function #429

Conversation

Jerry-Jinfeng-Guo
Copy link
Contributor

Added the interface function that returns the size of batch data, or raises error when the data is inconsistent.

tests/unit/test_utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
…ng use of existing utils function in validation.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
…ing.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo force-pushed the feature/DGC-2055-get-batch-size-function branch from 8aae5c2 to 3d77bdc Compare November 17, 2023 11:41
Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo marked this pull request as ready for review November 17, 2023 14:53
tests/unit/test_utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Show resolved Hide resolved
…atch size across components; removed `CConstDataset` interface, use `Dict[str, ndarray]` instead for sparse batch data creation; included `batch_data_array` size functionality and test cases; added interface import to the api reference.

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
src/power_grid_model/utils.py Outdated Show resolved Hide resolved
…nt`; etc

Signed-off-by: Jerry Guo <Jerry.Jinfeng.Guo@alliander.com>
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo merged commit a3b47b1 into PowerGridModel:main Nov 24, 2023
25 checks passed
@Jerry-Jinfeng-Guo Jerry-Jinfeng-Guo self-assigned this Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants