Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Variable node count bug #4

Closed
kkevlar opened this issue Aug 7, 2018 · 1 comment
Closed

Variable node count bug #4

kkevlar opened this issue Aug 7, 2018 · 1 comment
Labels
bug Something isn't working priority development priority simple A simple fix/enhancement

Comments

@kkevlar
Copy link
Contributor

kkevlar commented Aug 7, 2018

e96d0d5 added the ability to scale the density of the nodes along the trajectory with the = and - keys. However, scaling up the number of nodes and then scaling them back down causes the old, now unused nodes to stay where they were before the number of nodes was scaled down, causing an annoying bug.

These unused nodes need to be hidden in some way, or maybe this issue will be obsolete if nodes are defined differently in the xml as a fix for #1.
@kkevlar kkevlar added bug Something isn't working simple A simple fix/enhancement labels Aug 7, 2018
@kkevlar kkevlar mentioned this issue Aug 7, 2018
Closed
@kkevlar kkevlar added the priority development priority label Aug 7, 2018
@kkevlar
Copy link
Contributor Author

kkevlar commented Aug 8, 2018 •
edited
Loading

Fixed in bb77af1, however if #1 or #10 changes node definitions, this issue may need to be revisited.

@kkevlar kkevlar closed this as completed Aug 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working priority development priority simple A simple fix/enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kkevlar