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

[BUG] - AttributeError: 'int' object has no attribute 'check' in removeExtraSurfaces #250

Open
vtalanov opened this issue Jul 2, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@vtalanov
Copy link

vtalanov commented Jul 2, 2024

Describe the bug

I am using GEOUNED to convert a STEP-file to MCNP and to build a void using "simplify = voidfull" option.
Version 0.9.8.2 succefully decomposes the solids, builds the cells and then creates the void.
However, the version 1.0.1 (run over the same STEP-file using the same config,ini) fails to build complementary at some point (see below).

To Reproduce

See the Python script below used to run GEOUNED.
The only diffrerence would be in the second line:
sys.path.append('/home/l_talanov/GEOUNED-1.0.1')
instead of:
sys.path.append('/home/l_talanov/GEOUNED-0.9.8.2')

IMPORTANT
It does not look possible to isolate a single body since then the error during building void will disappear.

Expected behavior

GEOUNED version 1.0.1 completes building the void as the version 0.9.8.2 does.

Error message

Loop, Box to Split : 7 28
build complementary 7 0
build complementary 7 2
build complementary 7 4
build complementary 7 7
build complementary 7 9
build complementary 7 11
Traceback (most recent call last):
File "/home/l_talanov/Desktop/Data/07-02_v01g_re/1.0.1/myConvert_v1.py", line 10, in
GEO.Start()
File "/home/l_talanov/GEOUNED-1.0.1/GEOUNED/init.py", line 550, in Start
MetaVoid = Void.voidGeneration(
File "/home/l_talanov/GEOUNED-1.0.1/GEOUNED/Void/Void.py", line 43, in voidGeneration
voids = GetVoidDef(newMetaList, Surfaces, EnclosureBox, setting, Lev0=True)
File "/home/l_talanov/GEOUNED-1.0.1/GEOUNED/Void/Void.py", line 120, in GetVoidDef
cell, CellIn = z.getVoidComplementary(Surfaces, simplify=simplifyVoid)
File "/home/l_talanov/GEOUNED-1.0.1/GEOUNED/Void/VoidBoxClass.py", line 266, in getVoidComplementary
newSolid = removeExtraSurfaces(solDef, CTable)
File "/home/l_talanov/GEOUNED-1.0.1/GEOUNED/Utils/BooleanSolids.py", line 334, in removeExtraSurfaces
subCell.check()
AttributeError: 'int' object has no attribute 'check'

Please complete the following information):

  • OS: Linux Ubuntu 22.04
  • GEOUNED version: 1.0.1
  • Python version: 3.10.12

Additional context

config-ini.txt
myConvert_v1-py.txt
STEP-file of the problem can be provided bilaterally by request.

@vtalanov vtalanov added the bug Something isn't working label Jul 2, 2024
@shimwell
Copy link
Collaborator

shimwell commented Jul 2, 2024

Apologies for the changing API, the current versions of GEOUNED now make use of a JSON file format instead of the config.ini. Would it be possible to try again with the JSON or directly with the Python API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants