Skip to content

Commit 39cd417

Browse files
committed
Fix up QBX refinement mode changes
h/t @alexfikl x-ref #310 (review)
1 parent c1be40d commit 39cd417

3 files changed

Lines changed: 13 additions & 12 deletions

File tree

doc/conf.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
["py:class", r"optype.*"],
4949
["py:class", r"onp.*"],
5050
# sphinx >= 9.0 errors
51-
["py:class", r"_not_provided"],
51+
["py:class", r"NOT_PROVIDED"],
5252
["py:class", r"Callable\[.*"],
5353
]
5454

pytential/qbx/__init__.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -87,10 +87,6 @@
8787
.. autoclass:: NonFFTExpansionFactory
8888
8989
.. autodata:: FMMBackend
90-
91-
.. autoclass:: QBXRefinementMode
92-
93-
.. autoclass:: QBXRefinementNeededError
9490
"""
9591

9692

pytential/qbx/refinement.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -127,20 +127,25 @@ class QBXRefinementMode(Enum):
127127
Executing global QBX without refinement is unlikely to give
128128
accurate results.
129129
130-
.. attribute:: COMPLAIN
130+
.. attribute:: ERROR_ON_REFINEMENT
131131
132132
Do not perform any refinement, but raise a
133133
:class:`QBXRefinementNeededError` if stage-1 or stage-2 refinement
134134
would be required to satisfy the QBX refinement criteria.
135+
136+
.. note::
137+
138+
Refinement resulting from
139+
``force_stage2_uniform_refinement_rounds`` is still carried out.
135140
"""
136141

137142
REFINE = auto()
138143
NO_REFINEMENT = auto()
139-
COMPLAIN = auto()
144+
ERROR_ON_REFINEMENT = auto()
140145

141146

142147
class QBXRefinementNeededError(RuntimeError):
143-
"""Raised when :attr:`QBXRefinementMode.COMPLAIN` is in effect and
148+
"""Raised when :attr:`QBXRefinementMode.ERROR_ON_REFINEMENT` is in effect and
144149
refinement would be needed to satisfy the QBX refinement criteria.
145150
"""
146151

@@ -767,10 +772,10 @@ def _refine_qbx_stage1(lpot_source, density_discr,
767772
if iter_violated_criteria:
768773
violated_criteria.append(" and ".join(iter_violated_criteria))
769774

770-
if lpot_source.refinement_mode == QBXRefinementMode.COMPLAIN:
775+
if lpot_source.refinement_mode == QBXRefinementMode.ERROR_ON_REFINEMENT:
771776
raise QBXRefinementNeededError(
772777
"Stage-1 QBX refinement is needed but refinement mode is "
773-
f"'{QBXRefinementMode.COMPLAIN.name}'. "
778+
f"'{QBXRefinementMode.ERROR_ON_REFINEMENT.name}'. "
774779
"Criteria requiring refinement: "
775780
+ ", ".join(iter_violated_criteria))
776781

@@ -846,10 +851,10 @@ def _refine_qbx_stage2(lpot_source, stage1_density_discr,
846851
if iter_violated_criteria:
847852
violated_criteria.append(" and ".join(iter_violated_criteria))
848853

849-
if lpot_source.refinement_mode == QBXRefinementMode.COMPLAIN:
854+
if lpot_source.refinement_mode == QBXRefinementMode.ERROR_ON_REFINEMENT:
850855
raise QBXRefinementNeededError(
851856
"Stage-2 QBX refinement is needed but refinement mode is "
852-
f"'{QBXRefinementMode.COMPLAIN.name}'. "
857+
f"'{QBXRefinementMode.ERROR_ON_REFINEMENT.name}'. "
853858
"Criteria requiring refinement: "
854859
+ ", ".join(iter_violated_criteria))
855860

0 commit comments

Comments
 (0)