Skip to content
This repository has been archived by the owner on Nov 11, 2022. It is now read-only.

Adding Movers which can accept more than 3 points, and others #59

Merged
merged 17 commits into from
Nov 20, 2016
Merged

Adding Movers which can accept more than 3 points, and others #59

merged 17 commits into from
Nov 20, 2016

Conversation

Awlexus
Copy link
Contributor

@Awlexus Awlexus commented Nov 20, 2016

Improved my other movers, added a Sinus Mover and made a little optimization.

@yugecin yugecin merged commit 8dcfe96 into yugecin:master Nov 20, 2016
@yugecin
Copy link
Owner

yugecin commented Nov 20, 2016

notes:

  • I merged the polymovers into the existing mover list
  • There are now 2 linear movers: the normal one and the poly one, is there a major difference between them?
  • slidermovers are disabled for the polymovers
    Oh and I see you creates the sinusmover, but it isnt used anywhere
    Thanks

@Awlexus
Copy link
Contributor Author

Awlexus commented Nov 20, 2016

The Poly-linearmover actually just calls the standard one. It won't be necessary in the future, since I've made a generic Factory and mover, which just need a class to function like the original.

@yugecin
Copy link
Owner

yugecin commented Nov 24, 2016

I see you're still adding things but you haven't pulled in the meantime. I changed some important stuff inside the Dancer etc class.

@Awlexus
Copy link
Contributor Author

Awlexus commented Nov 24, 2016

I noticed that. I ran into some trouble while merging, so I will make a pull request once I've fixed it (again). This should also solve #79

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants