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

removed old command line scripts #3300

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

shimwell
Copy link
Member

Description

This PR attempts to remove scripts which are less useful now the Python API has these capabilities

Alternative to #3297 which attempts to update and move the scripts.

I think these scripts are not tested in the CI and one of them raises error when run

This would also ease packaging, reduce maintenance, etc

openmc-ace-to-hdf5 - suggest users make use of IncidentNeutron.from_ace
openmc-plot-mesh-tally - breaks on AttributeError: module 'matplotlib' has no attribute 'use', suggest users make use of openmc-plotter
openmc-track-combine - suggest users make use of openmc.Tracks.combine
openmc-track-to-vtk - suggest users make use of openmc.Tracks.write_to_vtk
openmc-update-inputs - perhaps this one should be ported to the python API or can be left?
openmc-update-mgxs - the input this converts has been deprecated for a while now, do we still need this?
openmc-voxel-to-vtk suggest users make use of openmc.voxel_to_vtk

Fixes # (issue)

Checklist

  • I have performed a self-review of my own code
  • I have followed the style guidelines for Python source files (if applicable)

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.

1 participant