From 22e23374dd2433903168dc5c09c9fc603c293527 Mon Sep 17 00:00:00 2001 From: Olek <45364492+yardasol@users.noreply.github.com> Date: Thu, 11 Aug 2022 13:38:45 -0500 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Paul Romano --- docs/source/pythonapi/deplete.rst | 2 +- openmc/deplete/openmc_operator.py | 1 - tests/regression_tests/deplete_no_transport/test.py | 4 +--- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/docs/source/pythonapi/deplete.rst b/docs/source/pythonapi/deplete.rst index 4154e35df..dd679a40a 100644 --- a/docs/source/pythonapi/deplete.rst +++ b/docs/source/pythonapi/deplete.rst @@ -43,7 +43,7 @@ algorithms `_. SILEQIIntegrator Each of these classes expects a "transport operator" to be passed. OpenMC -provides The following classes implement transport operators: +provides the following transport operator classes: .. autosummary:: :toctree: generated diff --git a/openmc/deplete/openmc_operator.py b/openmc/deplete/openmc_operator.py index c34065078..ee569bf71 100644 --- a/openmc/deplete/openmc_operator.py +++ b/openmc/deplete/openmc_operator.py @@ -534,7 +534,6 @@ class OpenMCOperator(TransportOperator): rates[i] = self._rate_helper.divide_by_adens(number) # Scale reaction rates to obtain units of reactions/sec - print(f"flux, power, or source rate: {self._normalization_helper.factor(source_rate)}") rates *= self._normalization_helper.factor(source_rate) # Store new fission yields on the chain diff --git a/tests/regression_tests/deplete_no_transport/test.py b/tests/regression_tests/deplete_no_transport/test.py index bf4a4b132..f5ee4bf1d 100644 --- a/tests/regression_tests/deplete_no_transport/test.py +++ b/tests/regression_tests/deplete_no_transport/test.py @@ -23,9 +23,7 @@ def fuel(): @pytest.fixture(scope="module") def micro_xs(): micro_xs_file = Path(__file__).parents[2] / 'micro_xs_simple.csv' - micro_xs = MicroXS.from_csv(micro_xs_file) - - return micro_xs + return MicroXS.from_csv(micro_xs_file) @pytest.fixture(scope="module") def chain_file():