adrianinsaval
commented
4 months ago If I'm not mistaken ifcopenshell is cross compiled for arm64 so the tests aren't run
Closed oursland closed 3 months ago
adrianinsaval
commented
4 months ago If I'm not mistaken ifcopenshell is cross compiled for arm64 so the tests aren't run
oursland
commented
4 months ago That would explain why the tests aren't being run. What I do not yet understand is why the .dylib packages necessary for the Python module are not installed in the conda package.
Krande
commented
3 months ago Hey, I just saw we were some missing build dependencies needed for cross-compilation according to this article. I have included them here in the PR -> https://github.com/conda-forge/ifcopenshell-feedstock/pull/50
Unfortunately I don't own a mac, so please let me know if either of you have the opportunity to help test the osx arm package before we make a new release?
Update:
I just merged a new release ifcopenshell=0.7.10 which hopefully should work as expected. Please let me know if you continue to see this issue?
Best Regards Kristoffer
Krande
commented
3 months ago Hey @oursland, it would help a lot if you could test ifcopenshell=v0.7.11 and see if this is still a problem?
oursland
commented
3 months ago Hey @oursland, it would help a lot if you could test
ifcopenshell=v0.7.11and see if this is still a problem?
I can confirm that the module now loads. A plain mamba install ifcopenshell==0.7.11 results in the following message on import:
Python 3.11.9 | packaged by conda-forge | (main, Apr 19 2024, 18:34:54) [Clang 16.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import ifcopenshell
No stream support: No module named 'lark'Running mamba install lark does resolve the issue. Is lark a runtime requirement?
Unfortunately there are dependency conflicts with the version of occt that ifcopenshell is built against in the application (FreeCAD), so I cannot test in that environment to ensure correct operation at this time.
Krande
commented
3 months ago That's good to hear!
Is lark a runtime requirement?
I believe lark is optional. But I don't see any harm in adding it as a runtime requirement. I'll consider adding lark as a runtime dependency in the next release in a few weeks
oursland
commented
3 months ago If it is optional, then I imagine it is unnecessary. If anything, too many dependencies lead to the problem I'm experiencing w.r.t. occt.
As the import is functional, I'll close the issue.
Solution to issue cannot be found in the documentation.
Issue
The necessary libraries for macOS ARM64 are not being built in the package and cannot be imported. This is reflected in the Azure builds, although it is not triggering an error. Perhaps the build scripts need to be updated.
In
meta.yamlthetestphase should be performing animport ifcopenshell. This is present in themacOSbuilds for x86-64, but not in themacOSfor ARM64.macOS x86-64 logs
TEST START: /Users/runner/miniforge3/conda-bld/osx-64/ifcopenshell-0.7.9-py311_all_h073491e_100.condais present and shows successful testing of importing theifcopenshelllibrary.macOS ARM64 logs
The testing phase is skipped, and attempts to import the library on a macOS ARM64 machine fails.
Installed packages
Environment info