Skip to content

Rubbers meshGen fix and Gizmo positioning fix for rubbers and MWG#384

Merged
freezy merged 3 commits into
freezy:masterfrom
Cupiii:Rubbers-Fix
Mar 13, 2022
Merged

Rubbers meshGen fix and Gizmo positioning fix for rubbers and MWG#384
freezy merged 3 commits into
freezy:masterfrom
Cupiii:Rubbers-Fix

Conversation

@Cupiii
Copy link
Copy Markdown
Collaborator

@Cupiii Cupiii commented Mar 12, 2022

Fixes Issue "Rubbers do not colllide sometimes #377".

Wrote Rubbers from scratch (copied some code from MWG).

Also fixed UV Mapping

@freezy
Copy link
Copy Markdown
Owner

freezy commented Mar 12, 2022

Cool! Is this something that could go back to VPX?

Copy link
Copy Markdown
Owner

@freezy freezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! There is a warning about _middlePoint, says it's never assigned but used. Can it be removed?

@Cupiii
Copy link
Copy Markdown
Collaborator Author

Cupiii commented Mar 12, 2022

Have to look at it tomorrow.

@Cupiii Cupiii changed the title Rubbers fix Rubbers fix and _middlePoint Fix for Rubbers and MWG Mar 13, 2022
@Cupiii Cupiii changed the title Rubbers fix and _middlePoint Fix for Rubbers and MWG Rubbers fix and Gizmo positioning fix for rubbers and MWG Mar 13, 2022
@Cupiii Cupiii changed the title Rubbers fix and Gizmo positioning fix for rubbers and MWG Rubbers meshGen fix and Gizmo positioning fix for rubbers and MWG Mar 13, 2022
@Cupiii
Copy link
Copy Markdown
Collaborator Author

Cupiii commented Mar 13, 2022

_middlepoint is set again for rubbers and MWG (where it was also not working). Changed PR Topic accordingly.

Test run fine on my computer.

@Cupiii
Copy link
Copy Markdown
Collaborator Author

Cupiii commented Mar 13, 2022

Oh, and yes, that may be backported to VP as well, but check for MWG branch, where I edited something in SplineVertex class. I think I only made the 2d spline non loopable, but i may have altered more without knowing. I should have used the 3dSpline Class which was used for Ramps... but did not...

@freezy freezy merged commit da4b8f8 into freezy:master Mar 13, 2022
@Cupiii Cupiii deleted the Rubbers-Fix branch March 13, 2022 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants