Skip to content

Commit 097e94d

Browse files
committed
Remove HauserParabolicSmoother as default smoother.
This removes HauserParabolicSmoother as a default smoother inside `prpy.base.robot`. Having `HauserParabolicSmoother` hard-coded in this `__init__()` means that _any_ `prpy` robot depends on the external package `or_parabolicsmoother`, even if they do not use it at all in their subclass implementation. Since we can easily override this default in subclassed (and almost always do to customize the retiming settings anyway), there is no reason to force this dependency.
1 parent 8d89442 commit 097e94d

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/prpy/base/robot.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
from ..tsr.tsrlibrary import TSRLibrary
3535
from ..planning.base import Sequence, Tags
3636
from ..planning.ompl import OMPLSimplifier
37-
from ..planning.retimer import HauserParabolicSmoother, OpenRAVEAffineRetimer, ParabolicRetimer
37+
from ..planning.retimer import OpenRAVEAffineRetimer, ParabolicRetimer
3838
from ..planning.mac_smoother import MacSmoother
3939
from ..util import SetTrajectoryTags
4040

@@ -71,10 +71,7 @@ def __init__(self, robot_name=None):
7171
# (joint simplificaiton and retiming).
7272
self.simplifier = None
7373
self.retimer = ParabolicRetimer()
74-
self.smoother = Sequence(
75-
HauserParabolicSmoother(),
76-
self.retimer
77-
)
74+
self.smoother = self.retimer
7875
self.affine_retimer = OpenRAVEAffineRetimer()
7976

8077
def __dir__(self):

0 commit comments

Comments
 (0)