multi_agent_path_planning icon indicating copy to clipboard operation
multi_agent_path_planning copied to clipboard

Saving to video currently does not work for lack of a canvas

Open notger opened this issue 2 years ago • 3 comments

See title.

You need to define a canvas before you define the function-animation. I have some fixed and slightly cleaned up code, so if you invite me to the repo, I can push a PR, if you want.

Way to replicate issue: Try creating video output. You will get an error message that says that an object related to canvas is None and does not have a certain property.

notger avatar May 18 '22 15:05 notger

You don't need to be a member of the repo to make a PR.

Can you please make fork the repo and then make a PR from that fork? Please check this out in case you aren't familiar with that work-flow

atb033 avatar May 18 '22 18:05 atb033

Thanks, so far never worked with forks. Will do.

notger avatar May 19 '22 08:05 notger

Here we go: https://github.com/atb033/multi_agent_path_planning/pull/11 .

notger avatar May 19 '22 08:05 notger

closing as the issue was not reproduced

atb033 avatar Oct 29 '22 09:10 atb033