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

New facehugger strain: Watcher #3726

Merged
merged 24 commits into from
Sep 19, 2023
Merged

Conversation

Hopekz
Copy link
Contributor

@Hopekz Hopekz commented Jun 26, 2023

Super easy to understand:

The watcher loses the ability to hide in exchange for letting them see further.

I don't have a sprite for this at the moment and probably won't for this PR so I have made them stand out by making them more orange because I think marines being able to see if a hugger is a Watcher might be important.
image

Currently they see 2 extra tiles like the runner but in testmerging I think it may change to 3 tiles (view range 10 instead of view range 9) unless I hit the perfect sweet spot already. EDIT: This number is now 3

the way I see it watcher is designed for 2 main playstyles:

1: Backlining to try to get someone not paying attention or busy (right behind marine frontline like medics doing medic stuff or someone resupplying/playing with inventory) or deep in the backline to try to find someone vulnerable.

2: Hanging around the xeno side frontline with increased situational awareness to be exactly where you need to be for xeno assisted caps. For example as a watcher during the TM I have an easier time watching a warrior fling someone and capitalizing a hug off a fling because I don't have to go out until the fling happens etc

update:

The activation of the view range now works similar to the runner; it is toggle-able
dreamseeker_tyE3nVA9V0

Fixed exploit that allowed watchers to hide despite not having the ability.

Update 2:

The watcher now does not take weeds over time.
Essentially implemented this:

Discord_9s19Vo2olV

Why:
The damage over time is meant so that the hugger cannot hide under some unweeded item the whole round to hug some random unsuspecting marine. The watcher cannot hide and is meant to be a backlining caste.
This allows the watcher to stalk people outside of weeds and wait for the perfect opportunity to hug when their target is distracted or busy.

Update 3:

Added my codersprite to the watcher.
These were created by me so hopefully a spriter or my spriter replaces these with a better one if they suck.

dreamseeker_O6I1iXuo21

🆑 Hopek
add: New facehugger strain: Watcher. The Watcher loses their ability to hide in exchange to see further and the ability to no longer take damage outside of weeds.
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -19. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@github-actions github-actions bot added Sprites Remove the soul from the game. Feature Feature coder badge labels Jun 26, 2023
@morrowwolf
Copy link
Member

Honestly, just keep the same sprite. That orange does not look good.

@morrowwolf morrowwolf marked this pull request as draft June 28, 2023 05:21
@Hopekz Hopekz marked this pull request as ready for review June 28, 2023 05:41
Copy link
Contributor

@sleepynecrons sleepynecrons left a comment

Choose a reason for hiding this comment

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

morrow already mentioned the orange and that got changed but please avoid just doing recolors in the future. it still needs a unique sprite though; something that changes up the silhouette and distinguishes itself from a normal facehugger at a glance.

maybe since they can't hide you could make them bigger and give them longer legs and some sunken areas for "eyes".

Copy link
Member

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

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

Conceptually this is approved. I believe the author wants some time to TM and play with how the zoom works which is fine. We are also waiting on a new sprite.

While waiting on the sprite feel free to ask for TM if you can convince a maintainer to keep an eye on it.

@morrowwolf morrowwolf added the Testmerge Candidate we'll test this while you're asleep and the server has 10 players label Jul 5, 2023
@Angoryskaven
Copy link

Found two bugs:

-1: Upon choosing the Watcher strain, if you are have toggled the "Hide" ability, it will carry over onto Watcher, allowing you to hide, at least for the first jump;

-2: When reconnecting, the Watcher's sight range is reverted to that of a normal hugger

@Hopekz
Copy link
Contributor Author

Hopekz commented Jul 13, 2023

update:
The activation of the view range now works similar to the runner; it is toggle-able
image

@Hopekz
Copy link
Contributor Author

Hopekz commented Jul 13, 2023

Found two bugs:

-1: Upon choosing the Watcher strain, if you are have toggled the "Hide" ability, it will carry over onto Watcher, allowing you to hide, at least for the first jump;

-2: When reconnecting, the Watcher's sight range is reverted to that of a normal hugger

1: Fixed

2: Fixed; it is now a toggle with a button like the runner so they can just toggle it again when reconnected.

@Hopekz
Copy link
Contributor Author

Hopekz commented Jul 13, 2023

Update 2:

The watcher now does not take weeds over time.
Essentially implemented this:

Discord_9s19Vo2olV

Why:
The damage over time is meant so that the hugger cannot hide under some unweeded item the whole round to hug some random unsuspecting marine. The watcher cannot hide and is meant to be a backlining caste.
This allows the watcher to stalk people outside of weeds and wait for the perfect opportunity to hug when their target is distracted or busy.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jul 15, 2023
@ihatethisengine ihatethisengine mentioned this pull request Aug 2, 2023
3 tasks
@harryob harryob added Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. and removed Testmerge Candidate we'll test this while you're asleep and the server has 10 players labels Aug 4, 2023
Copy link
Member

@harryob harryob left a comment

Choose a reason for hiding this comment

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

tm fine, needs sprite approval

@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Aug 12, 2023
@github-actions github-actions bot closed this Aug 20, 2023
@Huffie56 Huffie56 reopened this Aug 23, 2023
Copy link
Contributor

@sleepynecrons sleepynecrons left a comment

Choose a reason for hiding this comment

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

the current sprite is an improvement but it's just a cut and paste of the xenoid hugger head with no attempt at blending it into the watcher sprite. please fix. it also still needs a change in silhouette; maybe do something with the feelers/legs at the front considering they don't move in the running animation.

you could also do something with the tail but it will obviously require some animation work

example
image

@Hopekz Hopekz marked this pull request as draft September 1, 2023 23:24
@sleepynecrons sleepynecrons removed the Stale beg a maintainer to review your PR label Sep 1, 2023
@Hopekz
Copy link
Contributor Author

Hopekz commented Sep 18, 2023

Went ahead and removed the watcher sprite entirely because it didn't pass sprite review and I couldn't get it replaced in time for my trip to Poland.
Shame but it is what it is.

Thank you!

@Hopekz Hopekz marked this pull request as ready for review September 18, 2023 08:26
@morrowwolf morrowwolf added this pull request to the merge queue Sep 19, 2023
Merged via the queue into cmss13-devs:master with commit cc84139 Sep 19, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. Feature Feature coder badge Sprites Remove the soul from the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hugger Health Only Updates When Resting
9 participants