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.

Sign up for GitHub

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

Jump to bottom

Error message when plot filename path does not exist #2994

Closed
aprilnovak opened this issue May 8, 2024 · 0 comments · Fixed by #3038
Closed

Error message when plot filename path does not exist #2994

aprilnovak opened this issue May 8, 2024 · 0 comments · Fixed by #3038

Comments

@aprilnovak
Copy link
Contributor

Description

If you inadvertently provide a filename when generating plots, which does not exist on your machine (e.g., a typo in a full path), OpenMC will seg fault.

 Processing plot 1:
 /data/repo/outputs/analysis_pipeline/openmc/bol/hot_bol/plot_1.png...
Segmentation fault (core dumped)

Alternatives

Check if file path exists before trying to write there.

Compatibility

Improved user experience.
pitkajuh added a commit to pitkajuh/openmc that referenced this issue Jun 16, 2024
@pitkajuh
Fix openmc-dev#2994, non-existent path when saving plot causes segmen…
0c4dc3c 
…tation fault
@pitkajuh pitkajuh mentioned this issue Jun 16, 2024
Merged
5 tasks
church89 pushed a commit to openmsr/openmc that referenced this issue Jul 18, 2024
@pitkajuh @pshriwise
@paulromano @church89
Fix openmc-dev#2994, non-existent path causes segmentation fault when…
5a10426 
… saving plot (openmc-dev#3038)

Co-authored-by: Patrick Shriwise <[email protected]>
Co-authored-by: Paul Romano <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #2994, non-existent path causes segmentation fault when saving plot pitkajuh/openmc
1 participant
@aprilnovak