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

Engineers sometime refuse to capture actors under fog of war #16139

Closed
ghost opened this issue Jan 29, 2019 · 3 comments · Fixed by #16150
Closed

Engineers sometime refuse to capture actors under fog of war #16139

ghost opened this issue Jan 29, 2019 · 3 comments · Fixed by #16150
Assignees

Comments

@ghost
Copy link

ghost commented Jan 29, 2019

Reported by player eskimo on the forums:

Noticed a targeting issue with the engineer on top right oil of A Nuclear Winter. Engineers can't target Construction Yards through Fog of War, unless you have a unit there who is in view of it at the time of the Engineer clicking the Con Yard. I presume this is just because of the new Fog of War targeting vs MCV/ConYard code.

I can confirm both issues ( engineers refuse to capture conyard under fog / sometimes refuse to capture oil derricks). It seems the issue with oil derricks only appears if there are other actors in close proximity, like oil barrels on the mentioned map.

@netnazgul
Copy link
Contributor

netnazgul commented Jan 30, 2019

Is the first part related to "can't target oils when they are near the trees" bug? Because that's an old one I think noone really reported before.

@pchote pchote self-assigned this Jan 30, 2019
@pchote
Copy link
Member

pchote commented Jan 30, 2019

This is probably going to require #16048.

@pchote
Copy link
Member

pchote commented Jan 31, 2019

I have a work in progress fix on my fog-aware-enter branch. Capturing static structures works well, but mobile targets (hijacking) seems to be glitchier than on bleed. I will file a PR after i'm confident in the Hijacker behaviour and have ported most of the other Enter subclasses to use the new backend.

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

Successfully merging a pull request may close this issue.

2 participants