Skip to content

Conversation

KeithSloan
Copy link
Contributor

Add Energy related python libraries, as before but with new clean copy of ALLOWED_PYTHON_PACKAGES
to avoid merge conflict.

Libraries are
archaea
epw
ifc2osmd - Yes I know it has not been updated for a long time but it is starting point.
voxcity

@hyarion
Copy link

hyarion commented Aug 5, 2025

What are you planning to use them for? :)

@chennes
Copy link
Member

chennes commented Aug 31, 2025

I'm very concerned about the use of ifc2osmd-- this project is in its infancy, and does not provide any binary wheels, requiring compilation from source. This exposes our CI system (and anyone installing on their own system) to additional risks as it evaluates the package: native extension compilation is a code-execution vector. Compilers, linkers, and build scripts are invoked; a hostile setup.py/backend can run shell commands freely. I don't think we should allow packages without binary wheels.

@KeithSloan
Copy link
Contributor Author

KeithSloan commented Aug 31, 2025

If it cannot be installed securely then I will just close the pull request.

The reason I wanted to try it was it would give me a test case for osm data that I could compare with a different method.

I will just install manually on my system and use on a few ifc files to create some osm files.

@KeithSloan KeithSloan closed this Aug 31, 2025
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.

3 participants