comrob / Dubins3D.jl

Julia package to compute 3D Dubins Paths
BSD 2-Clause "Simplified" License
34 stars 6 forks source link

Duplicite code `Dlon.maneuver.case == "RLR"` #4

Open captain-eason opened 1 year ago

captain-eason commented 1 year ago

On line 140 of file dubinsmaneuver3d.jl, if Dlon.maneuver.case == "RLR" || Dlon.maneuver.case == "RLR" return [] end Why are the two judgment conditions in the if statement the same?

petvana commented 1 year ago

@captain-eason It seems like a typo, but I'll check that.

https://github.com/comrob/Dubins3D.jl/blob/a93544dc1b79972196b93af5bc278d7bb47954fa/src/dubinsmaneuver3d.jl#L140-L142

yfclark commented 5 months ago

@petvana this is a type error,RLR and LRL should not used in verticle plane,So try to construct should never include them when try to construct verticle dubin curve