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

Fix #11679: Facilities clipping issues #18358

Merged
merged 1 commit into from
Nov 5, 2022

Conversation

rik-smeets
Copy link
Contributor

@rik-smeets rik-smeets commented Oct 16, 2022

Adjust the bounding boxes to prevent clipping issues as shown in linked issue. I also didn't notice any regression when testing in busy parks with facilities.

Resolves #11679.

@Gymnasiast Gymnasiast self-requested a review October 18, 2022 21:36
@Gymnasiast Gymnasiast added the testing required PR needs to be tested before it is merged. label Oct 18, 2022
Copy link
Member

@Gymnasiast Gymnasiast left a comment

Choose a reason for hiding this comment

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

This breaks drawing peeps that are inside facilities:

afbeelding

@rik-smeets
Copy link
Contributor Author

rik-smeets commented Oct 19, 2022

This breaks drawing peeps that are inside facilities:

afbeelding

Good find, I completely forgot to test with peeps inside. I'll try to get those guys some privacy back ;-).

@rik-smeets
Copy link
Contributor Author

Turns out I also introduced some new clipping issues with nearby supports, and peeps walking inside the facilities didn't look proper anymore. Some more bounding box debugging seems to have resolved these issues, I even didn't need to change as much as before. Could you take another look @Gymnasiast?

Copy link
Member

@Gymnasiast Gymnasiast left a comment

Choose a reason for hiding this comment

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

This version fixes the peeps, but now the glitch in #1972 is back again:

afbeelding

@rik-smeets
Copy link
Contributor Author

This version fixes the peeps, but now the glitch in #1972 is back again:

afbeelding

Argh! How about changing this PR so it only resolves the other linked issue? #1972 is a lot harder and I could give that a separate go later.

@rik-smeets
Copy link
Contributor Author

I just changed the scope of this PR to solely resolve #11679. Can this get your approval now, @Gymnasiast ?

@rik-smeets rik-smeets changed the title Fix #1972/11679: Facilities clipping issues Fix #11679: Facilities clipping issues Nov 5, 2022
@Gymnasiast Gymnasiast merged commit c9b6292 into OpenRCT2:develop Nov 5, 2022
@Gymnasiast Gymnasiast added this to the v0.4.3 milestone Nov 5, 2022
@cheweytoo
Copy link

Awesome, that's much improved, thank you! There's just one flicker case left, for scenery in front of a stall:
From test park in #11679, in 4c218ba (a handful of commits ahead of this) on develop:

flickertree.mp4

@rik-smeets rik-smeets deleted the fix-1972-11679 branch November 5, 2022 13:41
@rik-smeets
Copy link
Contributor Author

@cheweytoo Yes, I'm quite sure that's the same issue as #1972. It's a tricky one, which I didn't manage to fix yet, unfortunately.

@cheweytoo
Copy link

Ah, I see. Well, fingers crossed then. :-)

@cheweytoo
Copy link

The privacy screen is directional I'm afraid:

unprivate_toilet

Doesn't seem to happen with every guest, and seems to happen more often with accessorised guests (hat, balloon), and only from the two "back view" directions, i.e. the directions that see the back of the stall.

unprivate_toilets.zip

Gymnasiast added a commit that referenced this pull request Nov 23, 2022
This reverts commit c9b6292.

Unfortunately #11679 had side effects more serious than the problem it set out to solve. So unfortunately we’ll have to revert it, unless @rik-smeets has another solution.

See here: #18358 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing required PR needs to be tested before it is merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

toilets clip through adjacent scenery under passing coaster train
3 participants