UCTE-DEF import/export: use IIDM Transformer ratedS - #4093
Draft
armic-artelys wants to merge 6 commits into
Draft
armic-artelys wants to merge 6 commits into
armic-artelys wants to merge 6 commits into
Conversation
armic-artelys
added this pull request to stack #4094
September 10, 2026 10:09
armic-artelys
force-pushed
the
feature/4073-ucte-export-rateds
branch
from
September 10, 2026 15:04
a8d694d to
b01a6cb
Compare
armic-artelys
force-pushed
the
feature/ucte-export-reporter
branch
from
September 10, 2026 16:04
36f6f4e to
7ee35a6
Compare
armic-artelys
force-pushed
the
feature/4073-ucte-export-rateds
branch
from
September 10, 2026 16:04
b01a6cb to
3ab8f3b
Compare
armic-artelys
changed the title
jeandemanged
marked this pull request as draft
September 11, 2026 14:58
jeandemanged
force-pushed
the
feature/ucte-export-reporter
branch
from
September 17, 2026 16:47
7ee35a6 to
f56444d
Compare
jeandemanged
force-pushed
the
feature/4073-ucte-export-rateds
branch
from
September 17, 2026 16:47
6c656b4 to
4b8186f
Compare
armic-artelys
force-pushed
the
feature/ucte-export-reporter
branch
3 times, most recently
from
September 18, 2026 09:01
c736ec6 to
bec6df5
Compare
armic-artelys
force-pushed
the
feature/4073-ucte-export-rateds
branch
2 times, most recently
from
September 21, 2026 08:09
6c656b4 to
3974907
Compare
armic-artelys
force-pushed
the
feature/4073-ucte-export-rateds
branch
from
September 25, 2026 09:47
3974907 to
e75a0fc
Compare
|
…power - Don't crash when nominal value can't be found in a transformer. Issue a warning in the report and dfault to 99999 (impossible value). Signed-off-by: Arthur Michaut <arthur.michaut@artelys.com>
…m ratedS Signed-off-by: Arthur Michaut <arthur.michaut@artelys.com>
armic-artelys
force-pushed
the
feature/4073-ucte-export-rateds
branch
from
October 1, 2026 14:14
9242ffd to
3a4f26f
Compare
jeandemanged
requested changes
Oct 2, 2026
…legacy property `nominalPower` Co-authored-by: Damien Jeandemange <damien.jeandemange@artelys.com> Signed-off-by: armic-artelys <arthur.michaut@artelys.com>
…sformer fields sourcing Co-authored-by: Damien Jeandemange <damien.jeandemange@artelys.com> Signed-off-by: armic-artelys <arthur.michaut@artelys.com>
Signed-off-by: Arthur Michaut <arthur.michaut@artelys.com>
jeandemanged
removed this pull request from stack #4094
October 2, 2026 13:17
Signed-off-by: Arthur Michaut <arthur.michaut@artelys.com>
jeandemanged
requested changes
Oct 2, 2026
| * Formats a double value the same way {@link com.powsybl.ucte.network.io.UcteWriter} exports it: as a decimal | ||
| * string truncated to {@code fieldLength} characters. | ||
| */ | ||
| private static String formatAsExported(double value, int fieldLength) { |
| import com.powsybl.commons.datasource.ResourceSet; | ||
| import com.powsybl.commons.test.AbstractSerDeTest; | ||
| import com.powsybl.iidm.network.*; | ||
| import com.powsybl.ucte.converter.util.UcteConverterConstants; |
jeandemanged
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is stacked on top of #4090.
Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
Fixes #4073.
What kind of change does this PR introduce?
Feature.
What is the current behavior?
UCTE importer and exporter use the deprecated property
nomimalPower(with a typo) in the IIDM 2w transformers.What is the new behavior (if this is a feature change)?
Property
nomimalPowerhas been replaced with the hardcoded fieldratedSsince v7.5.0. This PR replaces the use of the deprecated property with the new field.Retrocompatible: In the exporter, if the field ratedS is
NaN, we try to source it in the old way with thenomimalPowerprop.Absent nominal power: Nominal power is mandatory for ##T in UCTE-DEF, but it's not used in the PF. When nominal power is NaN in iidm (NaN field and absent, null or NaN prop), we issue a warning and default to
99999.Does this PR introduce a breaking change or deprecate an API?
What changes might users need to make in their application due to this PR? (migration steps)
Other information:
Ultimately, support any IIDM to UCTE-DEF conversion. See umbrella issue: #3989