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] - MCNP: cell references a null surface and contains other bounding surfaces #269

Closed
SterlingButters opened this issue Aug 9, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@SterlingButters
Copy link

Describe the bug

A clear and concise description of what the bug is. Remember to try to reproduce your bug with the latest version of the dev branch, the problem may already have been solved.

To Reproduce

Steps to reproduce the behavior:
for example:

  1. Generate csg.mcnp with default arguments
  2. Observe null surface (0) in cell definitions

IMPORTANT
Cannot provide problem parts.

Expected behavior

Didn't expect any surfaces with ID 0 since startSurf=1

Error message

MCNP: fatal error. cell <cell> references a null surface and contains other bounding surfaces.

Please complete the following information):

  • OS: Windows
  • GEOUNED version: 1.2.0
  • Python version: 3.11
@SterlingButters SterlingButters added the bug Something isn't working label Aug 9, 2024
@SterlingButters SterlingButters changed the title [BUG] - cell references a null surface and contains other bounding surfaces. [BUG] - MCNP: cell references a null surface and contains other bounding surfaces Aug 9, 2024
@psauvan
Copy link
Member

psauvan commented Aug 10, 2024

It is very difficult to found the reasons of the bug with so little information.
May be the CAD file has some features that FreeCAD doesn't handle correctly and produce this error in GEOUNED.
Try to open your CAD file with FreeCAD and check if it looks like your original CAD.

@SterlingButters
Copy link
Author

@psauvan Done - not seeing anything weird in the representation of the geometry. I was hoping there was some obvious explanation for why GEOUNED might be introducing a null surface (ID=0) e.g. inappropriate indexing. Is there a way for me to determine what surface in the geometry GOEUNED intended for the null surface (or is there another known reason for why GEOUNED would introduce a null surface)?

@akolsek
Copy link
Member

akolsek commented Aug 12, 2024

@SterlingButters, if you use SpaceClaim, try to check that your geometry doesn't have splines or that splines were not introduced while saving the geometry in a different format. I have seen a "surface ID 0" issue when a spline was present in the model. Removing the spline solved the issue. See issue 201.

@SterlingButters
Copy link
Author

SterlingButters commented Aug 12, 2024

@akolsek Yes that was the issue! The script in #201 helped me resolve the issue. I just didn't sort through the issues well enough. Closing the issue now - thanks!

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

3 participants