Skip to content

Commit

Permalink
Removing fictitious import (terrapower#906)
Browse files Browse the repository at this point in the history
  • Loading branch information
john-science authored and scottyak committed Oct 27, 2022
1 parent c126800 commit 9680df5
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 13 deletions.
16 changes: 4 additions & 12 deletions armi/physics/fuelCycle/fuelHandlerFactory.py
Expand Up @@ -31,18 +31,10 @@ def fuelHandlerFactory(operator):
fuelHandlerModulePath = cs["shuffleLogic"]

if not fuelHandlerClassName:
# User did not request a custom fuel handler.
# This is code coupling that should be untangled.
# Special case for equilibrium-mode shuffling
if cs["eqDirect"] and cs["runType"].lower() == RunTypes.STANDARD.lower():
from terrapower.physics.neutronics.equilibrium import fuelHandler as efh

return efh.EqDirectFuelHandler(operator)
else:
# give the default FuelHandler. This does not have an implemented outage, but
# still offers moving capabilities. Useful when you just need to make explicit
# moves but do not have a fully-defined fuel management input.
return fuelHandlers.FuelHandler(operator)
# give the default FuelHandler. This does not have an implemented outage, but
# still offers moving capabilities. Useful when you just need to make explicit
# moves but do not have a fully-defined fuel management input.
return fuelHandlers.FuelHandler(operator)

# User did request a custom fuel handler. We must go find and import it
# from the input directory.
Expand Down
2 changes: 1 addition & 1 deletion armi/utils/pathTools.py
Expand Up @@ -201,7 +201,7 @@ def cleanPath(path, mpiRank=0):
with full permissions on the cluster. You could accidentally delete everyone's work
with one misplaced line! This doesn't ask questions.
Safety nets include a whitelist of paths.
Safety nets include an allow-list of paths.
This makes use of shutil.rmtree and os.remove
Expand Down

0 comments on commit 9680df5

Please sign in to comment.