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

"Bend along surface field": Src U/V parameters are not documented good enough #4854

Closed
portnov opened this issue Jan 2, 2023 · 3 comments · Fixed by #5004
Closed

"Bend along surface field": Src U/V parameters are not documented good enough #4854

portnov opened this issue Jan 2, 2023 · 3 comments · Fixed by #5004
Assignees
@portnov

Comments

@portnov
Copy link
Collaborator

portnov commented Jan 2, 2023 •
edited
Loading

When Orientation == Z, Src U Min/Max for some reason correspond to min/max Y, not X!
@portnov portnov self-assigned this Jan 2, 2023
@portnov portnov changed the title "Bend along surface field": incorrect UV scale when domain of source surface is not [0;1] x [0;1] "Bend along surface field": Src U/V parameters are not documented good enough Jan 2, 2023
@satabol
Copy link
Collaborator

satabol commented Jul 8, 2023

Hello. Can you show example with nodes? (with parameters of surface node)

@satabol
Copy link
Collaborator

satabol commented Aug 24, 2023

If you move points (1) to points (2) of node "Surfaces->Plane(Surface)" then U/V swap.

image

So I think the problem is not in the node "Bend Along Surface field".

@satabol
Copy link
Collaborator

satabol commented Sep 27, 2023

Hi @portnov. May be convert default params for “Plane (surface)” to (2) to exclude this collision?

satabol added a commit that referenced this issue Sep 28, 2023
@satabol
fix #4854:
68f1f8c 
- Rename old class name to bl_idname: SvBendAlongSurfaceFieldNode -> SvExBendAlongSurfaceFieldNode
- create new version SvExBendAlongSurfaceFieldNodeMK2
@satabol satabol mentioned this issue Sep 28, 2023
Merged
satabol added a commit that referenced this issue Sep 28, 2023
@satabol
fix #4854
201ebd0 
Docs fixed.
satabol added a commit that referenced this issue Oct 4, 2023
@satabol
Merge pull request #5004 from nortikin/fix_4854_Bend_along_surface_fi…
fad7b2c 
…eld_Src_U_V_parameters

fix #4854: "Bend along surface field": Src U/V parameters are not documented good enough
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@portnov portnov

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@portnov @satabol