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 abnormality core system #2065

Merged
merged 1 commit into from
May 31, 2024

Conversation

vampirebat74
Copy link
Contributor

@vampirebat74 vampirebat74 commented Apr 19, 2024

About The Pull Request

Abnormality core! Or abnocores for short. This is a system that allows any abnormality spawned by abministrators or other unusual phenomena to reward the players for suppressing it. Specifically, any abnormality that was spawned without being assigned a console will spawn a core at its location after being Destroy()ed. Current rewards in mind would be: forcing the abnormality into an open containment cell, selling the core for cash or other goodies, or extracting E.G.O from the core directly. These may be more involved in alternative gamemodes.

While undecided at the time of writing, this can be globally disabled for alternate gamemodes such as R Corp Assault, where every abnormality could qualify for a core. They have relatively large icons, but have low layers and do not have collision.

image

Why It's Good For The Game

This simplifies the addition of many new mechanics without being too intrusive on regular gameplay.

2024-04-23.21-20-43.mp4

Changelog

🆑
add: adds the abnormality core subsystem
imageadd: adds default abnormality core icons
code: minor additions to _abnormality.dm and _info.dm
/:cl:

@vampirebat74 vampirebat74 marked this pull request as draft April 19, 2024 09:50
@Gboster-0
Copy link
Collaborator

probably shouldn't call it a subsystem in the title, as thats kinda taken by things like the SSLobotomyCorp and SSAtmos subsystems, the proper ones

@vampirebat74 vampirebat74 changed the title Adds the abnormality core subsystem Adds the abnormality core system Apr 19, 2024
@vampirebat74
Copy link
Contributor Author

probably shouldn't call it a subsystem in the title, as thats kinda taken by things like the SSLobotomyCorp and SSAtmos subsystems, the proper ones

You're right, and I didn't think about that. My bad

@Kitsunemitsu Kitsunemitsu added the Feature Something new being added. label Apr 20, 2024
@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 2 times, most recently from 278e940 to 85b6561 Compare April 22, 2024 02:52
@vampirebat74 vampirebat74 marked this pull request as ready for review April 22, 2024 02:53
LanceSmites328
LanceSmites328 previously approved these changes Apr 24, 2024
@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 2 times, most recently from 8fbfd13 to d0e203c Compare April 24, 2024 03:40
EgorDinamit
EgorDinamit previously approved these changes Apr 25, 2024
@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 3 times, most recently from 08ffaf0 to 1054edb Compare May 2, 2024 17:00
Copy link
Contributor

@Kitsunemitsu Kitsunemitsu left a comment

Choose a reason for hiding this comment

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

Hallucinations cause cores to spawn because they technically spawn an abnormality.
I want to let you know.

@vampirebat74
Copy link
Contributor Author

Hallucinations cause cores to spawn because they technically spawn an abnormality. I want to let you know.

This has been corrected

@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 3 times, most recently from 333e205 to 078d01d Compare May 19, 2024 21:01
@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 3 times, most recently from 310e972 to 1190f2e Compare May 27, 2024 07:30
@vampirebat74 vampirebat74 force-pushed the abnormality_cores branch 2 times, most recently from 74442e0 to 8f397be Compare May 30, 2024 20:56
Copy link
Contributor

@Coxswain-Navigator Coxswain-Navigator left a comment

Choose a reason for hiding this comment

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

Just a couple nitpicks, I believe this PR is mostly done at this point.

ModularTegustation/Teguicons/abno_cores/tool.dmi Outdated Show resolved Hide resolved
adds abno core icons

abno core extractor

final changes

Apply suggestions from code review

Co-authored-by: [̸R̵e̵d̴a̴c̶t̸e̸d̴]̵ <[email protected]>

requested changes

removes an unneeded comment

Adds skin prophet

Minor bug

Contract and book fixes

bug fix

fixes a hilarious bug where abnormalities spawned by hallucinations would leave cores behind

Fixes a bug with the naked nest

qoh core

what it says on the tin

fixes nobody is

RCA fixes

removes spontaneous core generation from R corp assault

icondiffbot issues

dmi changes

icondiffbot fix attempt

else if changes

else if change
Copy link
Contributor

@Coxswain-Navigator Coxswain-Navigator left a comment

Choose a reason for hiding this comment

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

This doesn't seem to break anything (anymore) and I have some pretty nifty expansions I can do with it. I'll be merging it.

@Kitsunemitsu Kitsunemitsu dismissed their stale review May 31, 2024 15:11

no longer relevant

@Coxswain-Navigator Coxswain-Navigator merged commit 6deeeea into vlggms:master May 31, 2024
8 checks passed
@vampirebat74 vampirebat74 deleted the abnormality_cores branch June 1, 2024 07:35
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.

6 participants