Skip to content
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

Update linux to 7.6.0 #95

Closed
wants to merge 6 commits into from
Closed

Update linux to 7.6.0 #95

wants to merge 6 commits into from

Conversation

frmdstryr
Copy link
Contributor

Description

Update binder/config to use occt 7.6.0 on linux.

Motivation and Context

For #94 , I don't have/use windows so someone else can fix that.

How Has This Been Tested?

DeclaraCAD with py3.8.

Screenshots (if appropriate):

occt-7 6 0

@frmdstryr frmdstryr closed this Dec 31, 2021
@trelau
Copy link
Owner

trelau commented Jan 2, 2022

btw..i'm working on a new (rewritten) version of the python binder (i.e., pyOCCT_binder). it's running locally on OCCT 7.6.0 on windows. will share the repo in coming days

@frmdstryr
Copy link
Contributor Author

Good to hear.. any chance it'll work with smesh too?

@trelau
Copy link
Owner

trelau commented Jan 3, 2022

@frmdstryr yes, in theory. the main motivation for redoing pybinder was to make it more robust and modular so it can (in theory) be applied to SMESH (as part of pySMESH project). at one point i had SMESH packaged into pyOCCT but it seems like those deserve to be separate things (tho it adds complexity in the binding tool). i could be convinced either way if you have an opinion on the matter.

@frmdstryr
Copy link
Contributor Author

frmdstryr commented Jan 5, 2022

I was playing around and got pySMESH 9.7 to compile with pyOCCT 7.6 but it seems like SMESH can no longer import base classes from OCCT. It's just giving a "referenced unknown base type" at import time for anything that extends something from OCCT. It's odd because I had it working with smesh 9.6... Idk if I messed something up or what changed. This wouldn't be a problem if it were all one library... but I still think they should be separate.

Edit: Nevermind, it works fine, I just missed a few types...

@Krande Krande mentioned this pull request Apr 22, 2022
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.

2 participants