Skip to content

Explicit parsing of fields of binary parameters#334

Merged
g-bauer merged 4 commits intomainfrom
fix_binary_parameter_parsing
Jan 23, 2026
Merged

Explicit parsing of fields of binary parameters#334
g-bauer merged 4 commits intomainfrom
fix_binary_parameter_parsing

Conversation

@g-bauer
Copy link
Contributor

@g-bauer g-bauer commented Jan 22, 2026

Due to flattened deserialization with serde in conjunction with an optional field inside BinaryRecord the current main silently transforms parsing errors into Nones. This PR solves this issue by explicitly parsing the record. Our current interfaces on the Python side require the inner model record of a binary parameter to be a struct.

@g-bauer g-bauer merged commit 80e9207 into main Jan 23, 2026
17 checks passed
@g-bauer g-bauer deleted the fix_binary_parameter_parsing branch January 23, 2026 16:20
prehner pushed a commit that referenced this pull request Jan 25, 2026
* Explicit parsing of fields of binary parameters to provide helpful error messages

* Added test for PR-parsing from JSON, added Python binding tests to GH action workflow, moved pyo3 extension feature to pyproject.toml

* Added changelog entries

* Small modification to changelog
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.

Peng-Robinson EOS does not appear to load BIPs from Parameters into EquationOfState (or missing docs on correct BinaryRecord format for PR)

2 participants