Skip to content

Commit

Permalink
Remove unused avoid_staging_area global variable
Browse files Browse the repository at this point in the history
This CL removes the declaration of the variable `avoid_staging_areas`
from route_planning, and removes the call to set this from full_demo.
Note no other scripts refer to this variable, and current route planning
logic does not use it.

Bug: #45, #39
Test: Passes all tests
Test Summary:          | Pass  Total     Time
ConstructionBots Tests |   15     15  3m48.2s
     Testing ConstructionBots tests passed
  • Loading branch information
tashakim authored Jan 30, 2024
1 parent 4b0f9e7 commit 9f5fb14
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
1 change: 0 additions & 1 deletion src/full_demo.jl
Original file line number Diff line number Diff line change
Expand Up @@ -558,7 +558,6 @@ function run_lego_demo(;
save_animation!(visualizer, "$(anim_prog_path)preprocessing.html")
end
end
set_avoid_staging_areas!(true)

execution_start_time = time()
status, time_steps = run_simulation!(env, factory_vis, anim, sim_params)
Expand Down
9 changes: 1 addition & 8 deletions src/route_planning.jl
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,7 @@ export

export
set_use_deconfliction,
use_rvo,
avoid_staging_areas,
set_avoid_staging_areas!
use_rvo

function set_use_deconfliction(deconflict_strategies)
if in(:RVO, deconflict_strategies)
Expand All @@ -34,11 +32,6 @@ end
global USE_RVO = true
use_rvo() = USE_RVO

global AVOID_STAGING_AREAS = false
avoid_staging_areas() = AVOID_STAGING_AREAS
function set_avoid_staging_areas!(val)
global AVOID_STAGING_AREAS = val
end
global STAGING_BUFFER_RADIUS = 0.0
staging_buffer_radius() = STAGING_BUFFER_RADIUS
function set_staging_buffer_radius!(val)
Expand Down

0 comments on commit 9f5fb14

Please sign in to comment.