compas-dev / compas_occ

COMPAS wrapper for the Python bindings of OCC
http://compas.dev/compas_occ/
MIT License
15 stars 7 forks source link

monkey patching for conversion from and to OCC <-> Compas #3

Closed jf--- closed 3 months ago

jf--- commented 3 years ago

hi @brgcode cool to catch up on all things OS, BRG & Compas...

Perhaps a convenient way to continue the conversation is by PR?

hereby some thoughts wrt monkey patching pythonocc & compas for smooth interoperability rather than inheriting, I think monkey patching seems pretty convenient here...

tomvanmele commented 3 years ago

sounds good @jf--- and indeed was nice to catch up. we should do that more often :)

tomvanmele commented 3 years ago

to further the discussion, i guess it would be better to focus on stuff that OCC can do for compas behind the scene, rather than the explicit conversions i was experimenting with. the latter are good for understanding how things work but they are not very useful. tried something similar for a sphere as for the box but that clearly doesn't work the same way. can imagine this is like that in most cases...

some ideas

would it make sense to use the control points, knots, ... from rhino to recreate geometries here and compare the results of the above operations?