powsybl / pypowsybl

A PowSyBl and Python integration based on GraalVM native image
Mozilla Public License 2.0
52 stars 10 forks source link

Windows CI fixed and add basic import test step in CI #760

Closed EtienneLt closed 1 month ago

EtienneLt commented 1 month ago

Please check if the PR fulfills these requirements

Does this PR already have an issue describing the problem?

What kind of change does this PR introduce?

What is the current behavior?

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change or deprecate an API?

If yes, please check if the following requirements are fulfilled

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

obrix commented 1 month ago

Maybe linked to the following issue : https://github.com/actions/runner-images/issues/10020

EtienneLt commented 1 month ago

it seems to work

obrix commented 1 month ago

Yes, I think we will be able to remove the workaround once they fix their runner image. Also I added a simple standalone_test.py that only import powsybl and print version to run before the full tests. That was just to debug at first but I am wondering if it could be a good idea to let it in place to be able to quickly see if an issue is related to the whole import and init of pypowsybl or a to a specific test. Any opinion @EtienneLt @geofjamg ?

obrix commented 1 month ago

It looks like the windows image has been fixed.