Skip to content

Conversation

@kshitij-maths
Copy link
Member

This PR fixes #285

  • Updated: pygem/cad/cad_deformation.py: Replaced deprecated global function calls (e.g., IGESControl_Controller_Init) with static class methods (e.g., IGESControl_Controller.Init) to support recent pythonocc-core versions.
  • Verification: Test passed
python -m pytest tests/test_ffdcad.py 
=====================test session starts=====================
platform linux -- Python 3.12.12, pytest-9.0.1, pluggy-1.6.0
rootdir: /mnt/raid/kpandey/Kshitij/ITCS/PyGeM
configfile: pyproject.toml
plugins: anyio-4.11.0, cov-7.0.0
collected 2 items                                                                                                                                                           

tests/test_ffdcad.py  .. [100%]

==================warnings summary==========================
<frozen importlib._bootstrap>:488: 52 warnings
  <frozen importlib._bootstrap>:488:
 DeprecationWarning: builtin type SwigPyPacked has no
 __module__ attribute

<frozen importlib._bootstrap>:488: 52 warnings
  <frozen importlib._bootstrap>:488:
 DeprecationWarning: builtin type SwigPyObject has no
 __module__ attribute

<frozen importlib._bootstrap>:488: 52 warnings
  <frozen importlib._bootstrap>:488:
 DeprecationWarning: builtin type swigvarlink has no
 __module__ attribute

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=================2 passed, 156 warnings in 1.42s====================

@ndem0
Copy link
Member

ndem0 commented Dec 5, 2025

Thanks @kshitij-maths! But the error is still there on macOS system!
There are also conflicts; please rebase them in order to merge the PR

@kshitij-maths kshitij-maths force-pushed the fix/iges-test-and-deprecations branch from 09e0122 to bfd1522 Compare December 5, 2025 10:33
@kshitij-maths
Copy link
Member Author

@ndem0 resolved.

@kshitij-maths kshitij-maths self-assigned this Dec 5, 2025
@ndem0 ndem0 merged commit b482f27 into mathLab:master Dec 5, 2025
13 checks passed
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.

Error on test_ffd_iges_pipe_mod_through_topods_shape

2 participants