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

Adds the mining breach type #2615

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

Conversation

vampirebat74
Copy link
Contributor

@vampirebat74 vampirebat74 commented Dec 17, 2024

About The Pull Request

What it says on the tin. This breach type is primarily used in the Enkephalin Rush gamemode, though it is also used by monoliths that may appear elsewhere in the future. The main appeal of this breach type is that it prevents most forms of teleportation on the initial breach, as well as addressing gameplay issues related to mining threats.

A few abnormalities are "disabled" through this by deleting themselves immediately. This mainly applies to superbosses such as whitenight and distortedform, whos cores will be used to present an optional fight instead. (TODO: reduce the monetary value of these abnormality cores to zero)

In addition, the scope of this PR includes new breaches entirely, which are planned for meat lantern and rose sign. Their current mechanics impose unreasonable challenges due to the nature of the gamemode this PR affects.

A handful of abnormalities whom lack regular breaches have also been given abilities for this specific breach type. Those who are inanimate objects like A Poor Screenwriter's Note and Book Without Pictures or Dialogue are now unable to move or attack adjacent targets. Additionally, many abnormalities who use this feature to summon one or more hostile units have been given damage coefficients and increased HP.

Singing Machine will now perform its music until suppressed, boasting high HP and decent defenses.

Three "summoner" archetypes have been created.

Type 1:
Immobile, tanky unit that summons many units over time, until suppressed.
YMBS, Book, Rose Sign

Type 2:
Controls a subordinate entity
La Luna, Screenwriter

Type 3:
Capable of combat and movement, slower summons and lesser quantities permitted.
Laetitia, Fairy Festival, Contract

Bloodbath now has a unique hostile breach recycling assets from its Distorted Form variant.

ZAYIN abnormalities will utilize this breach type for an easier version of their respective pink midnight breaches, excluding Fairy Festival who has a unique breach.

Why It's Good For The Game

LC13 has normally imposed many restrictions on how abnormalities should behave for the main gameplay loop. I wish for my gamemode to eventually include every abnormality in the game, each imposing a challenge to the player. I hope that these new features will be used for more creative projects in the future.

Changelog

🆑
add: breach_mining to abnormalities defines
fix: rose sign's roses now remove statuses when qdeleted
imageadd: fairy festival breach
soundadd: fairy queen sfx, one sin attack
code: new breach types, monolith changes
/:cl:

@vampirebat74 vampirebat74 marked this pull request as draft December 17, 2024 06:36
@Kitsunemitsu Kitsunemitsu added the Feature Something new being added. label Dec 17, 2024
@vampirebat74 vampirebat74 force-pushed the breach_mining branch 3 times, most recently from feda92b to 8ce18a3 Compare December 24, 2024 06:59
HE level

he, teth, waw

linters, some zayins

rose sign stuff

summoners + singing machine

fairy festival changes
@vampirebat74 vampirebat74 marked this pull request as ready for review January 1, 2025 07:47

/mob/living/simple_animal/hostile/abnormality/laetitia/proc/SummonAdds()//Mining breach summon
summon_cooldown = world.time + summon_cooldown_time
if(LAZYLEN(summon_list) > 9)//this list is not subtracted when minions are killed. Limited to 10 per breach
Copy link
Collaborator

Choose a reason for hiding this comment

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

"this list is not subtracted when minions are killed. Limited to 10 per breach" Whats the point of having a list then???
Replace this with a number, this is not only wastefull and can confuse coders but also makes the mobs hard-delete themselfes due to them having a reference being kept

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Something new being added.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants