-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Raise error or warning, if pythonfmu/resources/binaries/linux64/libpythonfmu-export.so is missing #129
Comments
I agree that something like this could be implemented. However, I have no idea how realize it. |
setuptools offers the possiblity to call commands before and after the packaging see pull request 136 |
The solution introduced by @jschueller in PR 188 breaks the pythonfmu functionality when running the python code from inside the FMU. This is because the path to the binaries is different depending on the context. The Running this code from inside the FMU will make the model fail while instantiating as the binaries are located in a different path within the FMU. Note: When the FMU is simulated and the python code inside of it is run, it will use the pythonfmu code that is bundled into the FMU as opposed to the installed pythonfmu package that you usually find in your python environment. Therefore, we shouldn't check for the binaries to be in the We (DNV) need to release a new version of pythonfmu for a project, so I'll be deleting this check from the |
fine, I can also check the binary from outside pythonfmu for now |
I installed pythonfmu by cloning this repository and using the provided
setup.py
script. On the first try I did not generatepythonfmu/resources/binaries/linux64/libpythonfmu-export.so
. This led to unusable FMUs. I think it would be helpful to raise a error or warning, if a user tries to install pythonfmu viasetup.py
andlibpythonfmu-export.so
is missing.Ideally one could entirely avoid running
cmake
andmake
and do everything insetup.py
, but I have no idea whether this is possible, at all.Current behavior (without
libpythonfmu-export.so
)no
libpythonfmu-export.so
in~/.local/lib/python3.8/site-packages/pythonfmu/
Current behavior (with
libpythonfmu-export.so
)if I run
cmake
etc. there is alibpythonfmu-export.so
in~/.local/lib/python3.8/site-packages/pythonfmu/
Expected behavior (without
libpythonfmu-export.so
)Add a warning (or even abort), if there is no
libpythonfmu-export.so
.The text was updated successfully, but these errors were encountered: