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.

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

In "Distribute Along Path", "Align to Path" parameter only affect the Y axis #13

Closed
HungryProton opened this issue Dec 28, 2020 · 7 comments
Labels
enhancement New feature or request

Comments

@HungryProton
Copy link
Owner

No description provided.

@HungryProton HungryProton added the enhancement New feature or request label Dec 28, 2020
@addmix
Copy link
Contributor

addmix commented Jan 4, 2021

I could see a use case for only rotating along the Y axis, keeping it as an option would be greatly appreciated

@jasonhowelldev
Copy link

I would love the option to toggle rotation for each axis separately.

@addmix
Copy link
Contributor

addmix commented Jan 11, 2021

I would love the option to toggle rotation for each axis separately.

I can work on implementing that functionality tomorrow.

@HungryProton
Copy link
Owner Author

I can work on implementing that functionality tomorrow.

I have a branch where I started this feature but never finished it. I can push it if you want a starting point, but I don't remember why I gave up midway.

@HungryProton
Copy link
Owner Author

HungryProton commented Jan 11, 2021

I pushed what I started on the branch issue/13

There are three options:

  • align_y_axis rotates the object, so the forward vector faces the curve direction.
  • align_x_axis should take care of the up and down rotation but this doesn't work at all.
  • align_z_axis takes care of the tilt by rotating around the Z axis but this is not supported as there's no way to change the curve's tilt from the editor anyway.

By the way, the fence object in the example scene is aligned on its x-axis so by default the alignment will be wrong. That's easy to solve with the Add Offset modifier by adding a local 90° rotation. This breaks as soon as tilt is introduced, so I wonder if we shouldn't simply ignore that tilt exists, or find another solution.

@addmix
Copy link
Contributor

addmix commented Jan 11, 2021

fixed in #28

@jasonhowelldev
Copy link

I just tried the fix and it appears to be working correctly to me. Thanks for working on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants