Fix error when position bounds are negative.

JuliaRobotics · Oct 11, 2018 · ee4abe2 · ee4abe2
1 parent 07ca12e
commit ee4abe2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/manipulate.jl b/src/manipulate.jl
@@ -28,8 +28,9 @@ function sliders(joint::Joint, values=clamp.(zeros(num_positions(joint)), positi
                  resolution=0.01, prefix="")
     map(bounds, labels, values) do b, label, value
         num_steps = ceil(Int, (upper(b) - lower(b)) / resolution)
+        r = range(lower(b), stop=upper(b), length=num_steps)
         slider(range(lower(b), stop=upper(b), length=num_steps),
-               value=value,
+               value=clamp(value, first(r), last(r)),
                label=string(prefix, label))
     end
 end

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -104,6 +104,14 @@ vis = Visualizer()
         widget = manipulate!(mvis)
     end
 
+    @testset "manipulation - negative bounds issue" begin
+        mechanism = rand_chain_mechanism(Float64, Revolute{Float64})
+        joint = first(joints(mechanism))
+        joint.position_bounds .= RigidBodyDynamics.Bounds(-2.0, -1.0)
+        mvis = MechanismVisualizer(mechanism)
+        widget = manipulate!(mvis)
+    end
+
     @testset "examples" begin
         dir = joinpath(@__DIR__, "..", "examples")
         for file in readdir(dir)