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

[Bug]: Cover VBL Island Edge Case #4437

Closed
ColdAnkles opened this issue Nov 20, 2023 · 3 comments · Fixed by #4438
Closed

[Bug]: Cover VBL Island Edge Case #4437

ColdAnkles opened this issue Nov 20, 2023 · 3 comments · Fixed by #4438
Assignees
Labels

Comments

@ColdAnkles
Copy link
Contributor

Describe the Bug

Took this for a spin, there's one edge case that is not working as I would expect: a token in cover will have its vision blocked by the opposite side of an enclosed ocean (hole):
image

Compare with the case where the "hole" is not completely enclosed, then vision is not obstructed at all:
image

I would expect the first case to also not block vision either.

Originally posted by @kwvanderlinde in #4299 (comment)

To Reproduce

Create an island of cover VBL like so:
image

Expected Behaviour

The far side of the cover should be blocking vision - be unable to see back into itself

Screenshots

No response

MapTool Info

No response

Desktop

No response

Additional Context

No response

@kwvanderlinde
Copy link
Collaborator

My quoted comment actually stated the opposite expected behvaiour: if a token is in an island of Cover VBL, the island should not block its vision.

I have in mind Hill VBL, which is the only other VBL case where LOS can start in an island, cross an ocean, and then encounter the same island. For Hill VBL, the second encounter is treated the same as the first, i.e., no blocking is applied. I think it should be the same for Cover VBL.

But I'm open to having my mind changed 😄 Maybe Hill VBL is in the wrong here ...

@ColdAnkles
Copy link
Contributor Author

ColdAnkles commented Nov 21, 2023

I see it as you're in a shrubbery that you can see out of - but that can't be seen into. If it's a hedge shaped shrubbery and you're looking across an open area to the other end of the hedge, you shouldn't be able to see back into it. Something like the leaves form a one-way view inhibitor, and the branches and so on inside let you see whats going on normally.

I think Hill is fine, without some fancier raycasting or elevation systems - if you're on a hill, you can see the tops of all other hills at the same height - should really block at the back edge if and only if a ray cast passes over the front edge first - no idea if even possible with how it's done in Maptool.

kwvanderlinde added a commit to kwvanderlinde/maptool that referenced this issue Nov 23, 2023
- Fix RPTools#4437 more directly by not excluding the current island while iterating over siblings.
- Fix facing of current ocean, while removing redundant null check against the parent.
kwvanderlinde added a commit to kwvanderlinde/maptool that referenced this issue Nov 23, 2023
- Fix RPTools#4437 more directly by not excluding the current island while iterating over siblings.
- Fix facing of current ocean, while removing redundant null check against the parent.
kwvanderlinde added a commit to kwvanderlinde/maptool that referenced this issue Nov 23, 2023
- Fix RPTools#4437 more directly by not excluding the current island while iterating over siblings.
- Fix facing of current ocean, while removing redundant null check against the parent.
kwvanderlinde added a commit to kwvanderlinde/maptool that referenced this issue Nov 23, 2023
- Fix RPTools#4437 more directly by not excluding the current island while iterating over siblings.
- Fix facing of current ocean, while removing redundant null check against the parent.
@kwvanderlinde
Copy link
Collaborator

Tested on 1.15.0-rc.2. Behaving as expected - it is no longer possible to see back into Cover VBL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging a pull request may close this issue.

2 participants