API stability documentation
Hi all, We have discussed some time ago the possibility of flagging some features with some kind of "experimental" or "not stable" flag, but did not decided anything. I have come upon the api guardian project, which provides the @API annotation and which is used for example in…
Sharing test code between iidm-impl and powsybl-network-store (and possibly…
Hi, we already have two implementations of the iidm API. We should find a solution to be able to share most of the test code for these implementations. Currently, the test code lives in iidm-impl and has about 50 classes. Does anyone have an idea on how to do this ? …
Working variant ID in computations SPI (load flow in particular)
The new load flow SPI requires the implementation to handle a working variant as its argument. I think that the handling of this argument should be common for all implementations and should therefore be removed from the SPI and included in the LoadFlow.Runner class. Another…
Open load flow Executor
The open loadflow uses the default Executor for its execution : shouldn't it use the Executor provided by the ComputationManager instead? Otherwise, how can the user specify which Executor he wants to use for the computation ? From a broader point of view, shouldn't we encourage…
Naming strategy for modules and packages in powsybl-core
Some of you may already know it but one objective for the next release is to homogeneize the naming strategy for powsybl-core modules and packages. To start a discussion about these changes I have created a file summing up a naming strategy I elaborated from existing standards…
Shouldn't we use Set instead of List for PostProcessors names in ImportConfig?
To my understanding, postProcessors are supposed to have unique names. Shouldn't we then use Set instead of List? This remark is also relevant for ImporterWrapper (in Importers) and ImportersLoader. If you agree, I will write an issue in this sense.
Adding dependency:analyze to build checks
Maven can fail the build if there are used but undeclared dependencies or unused but declared dependencies. https://maven.apache.org/plugins/maven-dependency-plugin/analyze-only-mojo.html Is anyone against enabling this check, once all the current problems are fixed ?
How should we pull in an implementation of jaxb-api
in Java11+, the jre doesn't bundle an implementation of jaxb-api, so we need to decide on our own how to depend on it. Currently, powsybl-core/commons and powsybl-core/simulation-api call methods from jaxb-api. powsybl-core/commons has a compile dependency to jaxb-api and…