Skip to content

feat: serialize to JSON string instead of file#223

Merged
vincentkoppen merged 11 commits into
mainfrom
pgm/serialize-json-string
May 7, 2026
Merged

feat: serialize to JSON string instead of file#223
vincentkoppen merged 11 commits into
mainfrom
pgm/serialize-json-string

Conversation

@jaapschoutenalliander

@jaapschoutenalliander jaapschoutenalliander commented Apr 10, 2026

Copy link
Copy Markdown
Member

This pull request adds support for serializing and deserializing grid objects to and from JSON strings (in addition to files)

fixes: #210

jaapschoutenalliander and others added 6 commits April 4, 2026 19:32
Signed-off-by: Jaap Schouten <jaap.schouten@alliander.com>
Signed-off-by: Jaap Schouten <jaap.schouten@alliander.com>
Signed-off-by: Jaap Schouten <jaap.schouten@alliander.com>
Signed-off-by: Jaap Schouten <jaap.schouten@alliander.com>
Signed-off-by: Jaap Schouten <jaap.schouten@alliander.com>
@jaapschoutenalliander

Copy link
Copy Markdown
Member Author

@TomInfiniot, we have set up a PR which gives to option to serialize to json_string. Does this also suit your needs?

Comment thread src/power_grid_model_ds/_core/model/grids/base.py Outdated

@Thijss Thijss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vincentkoppen vincentkoppen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@sonarqubecloud

sonarqubecloud Bot commented May 7, 2026

Copy link
Copy Markdown

@vincentkoppen vincentkoppen merged commit 2b86aef into main May 7, 2026
15 checks passed
@vincentkoppen vincentkoppen deleted the pgm/serialize-json-string branch May 7, 2026 14:57
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.

[IMPROVEMENT] Add serialize to dict & deserialize from dict option

3 participants