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

Misplaced swamp portal #56

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Misplaced swamp portal #56

wants to merge 2 commits into from

Conversation

szapp
Copy link
Collaborator

@szapp szapp commented May 9, 2021

There's a portal in the swamp that renders the water invisible at some point.

@AmProsius
Copy link
Owner Author

This issue was fixed in freddy's world patch.

@AmProsius AmProsius added the type: world bug This issue requires editing properties of objects in the game world. label Jan 9, 2021
@szapp szapp added the type: revert on save The fix for this issue impacts the game and should be reverted when saving. label Jan 24, 2021
@szapp
Copy link
Collaborator

szapp commented Jan 24, 2021

Are there details of this issue available? Like screenshots and especially the name of the portal (rooms)?

@AmProsius
Copy link
Owner Author

You can clearly see the issue at OW_PATH_BLOODFLY12_SPAWN01. There's a chunk of swamp missing.

056

@AmProsius AmProsius added the validation: validated This issue is still present even with Systempack/Union. label Jan 28, 2021
@szapp szapp removed the type: revert on save The fix for this issue impacts the game and should be reverted when saving. label Mar 1, 2021
@szapp szapp self-assigned this Mar 19, 2021
@szapp szapp added compatibility: difficult This issue is difficult to make compatible. impl: unknown There is no clear plan on how to implement this issue yet. labels Apr 5, 2021
@szapp
Copy link
Collaborator

szapp commented Apr 5, 2021

This fix will require a bit of studying: Is the only solution to remove the portal all together? Or can the portal be adjusted?

If removed, it has to be recreated and added to the world and portal list on every save.

@szapp
Copy link
Collaborator

szapp commented May 9, 2021

Name of the portals(!) is P:PSU01_.

This is early work in progress. The correct polygons have been
identified. They are currently only visually highlighted (painted red),
but there is no way yet to remove/modify the portal.

Refs #56
@szapp szapp marked this pull request as draft May 9, 2021 22:37
@szapp
Copy link
Collaborator

szapp commented May 13, 2021

My thought now is whether the portal cannot be changed at run-time but only while the world BSP tree is created. That would mean, hooking the constructors to prevent creating the portals to the sector "PSU01" (or the sector itself?). That would require some assembly code hooking (for performance) and might turn this fix into a session fix.

@szapp
Copy link
Collaborator

szapp commented Jan 14, 2022

Let’s check if the (formerly called?) Union primary patch (part of a basic Union installation) doesn’t already fix this problem. I read something about this several times.

@szapp szapp added validation: required This issue needs validation from one of the validators. and removed validation: validated This issue is still present even with Systempack/Union. labels Jan 14, 2022
@szapp
Copy link
Collaborator

szapp commented Jan 15, 2022

This is the relevant portion of the Union.patch file (lines 1428-1432). I haven't yet looked up what exactly is being done there. Nevertheless, it does not seem to fix this issue here. Maybe it addresses another issue.

    #patch [Disable portal in the swamp]
        HEX @0x00521086 = 'E9 B5 87 21 00'
        HEX @0x0051C9B6 = 'E9 B4 CE 21 00'
        HEX @0x00739836 = '8D 4C 24 78 51 E9 4B 78 DE FF 8B 4D 08 81 39 50 53 55 30 75 EB 8D 4C 24 78 51 C7 44 E4 34 45 3E 10 4E C7 44 E4 28 85 D8 0C 4E E9 36 78 DE FF 8D 4C 24 5C 51 E9 4C 31 DE FF 8B 4E 18 8B 49 18 81 79 04 53 55 30 31 75 E7 81 39 50 3A 5F 50 75 DF 8D 4C 24 5C 51 C7 44 E4 2C 45 3E 10 4E C7 44 E4 24 85 D8 0C 4E E9 2B 31 DE FF'
    #/patch

@AmProsius
Copy link
Owner Author

I've seen something like that in the changelog, but also realized that it didn't fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility: difficult This issue is difficult to make compatible. impl: unknown There is no clear plan on how to implement this issue yet. type: world bug This issue requires editing properties of objects in the game world. validation: required This issue needs validation from one of the validators.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants