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

[Need Feedback] TG Slaughter Demon Port Remake, THE REGIBBINGING #1387

Merged
merged 35 commits into from Aug 7, 2015

Conversation

Aurorablade
Copy link
Contributor

So i did a thing..and it messed up alot of other things..so i remade the
thing..
Adjusted some spans and redid the spawn locs based on the ghostie antag

So a remake of the branch i Effed up at #1323

Ports TG Slaughter Demon by @KorPhaeron tgstation/tgstation#9973 (is it bad i have been stalking this for days to port it?) All credits to him for the concept and more credits i shoved in the file itself.

New side event/antagonist. Simple animal that can use pools of blood as
portals to move around the station. Spooky scary.

-CtrlClick on an adjacent blood pool (or turf with blood on it) to
"enter" (start an infinite duration jaunt
-CtrlClick on a blood pool while jaunted to exit at that pool
-If you're pulling a mob while you enter a pool, the mob gets pulled in
with you, and you devour them, healing yourself
-Night vision

Adds an event (admin only) that spawns one.

Adds wizard item for summoning it. Has objective to kill the wizard (and everyone else).

Bonus additions from me: Crabrender and Clownrender. I also wanted to add a Ponyrender..but i would be lynched.

To do:
What more fluff can i add?
Does anyone want this as a random event?
Anything else to be done/added?

So i did a thing..and it messed up alot of other things..so i remade the
thing..
Adjusted some spans and redid the spawn locs based on the ghostie antag
@Aurorablade Aurorablade changed the title LETS DO THE GIT WARP AGAIN! LETS DO THE GIT WARP AGAIN! [DNM] TG Slaughter Demon Port Remake, THE REGIBBINGING Jun 25, 2015
@Fox-McCloud Fox-McCloud added Do Not Merge This PR must not be merged or closed Feature This PR is a new addition to the game labels Jun 25, 2015
NOTHING TO SEE HERE
Apprently travis need this file...
@TheDZD
Copy link
Contributor

TheDZD commented Jun 25, 2015

Might want to look at tgstation/tgstation#10151.

@Aurorablade
Copy link
Contributor Author

@TheDZD Will do.

fixes as per tgstation/tgstation#10151 Also
spread the one per limit to the Scrying orb, Can remove or add to other
items as suggested/needed by feedback.
@Aurorablade
Copy link
Contributor Author

So any feedback? Bits of fluff i can add? Make it an event?

Maybe have people explode into another demon and/or a demonic heart inspired by http://www.scp-wiki.net/scp-058 if they eat the heart item? (I know 'why would they do that?!' ....People are odd okay)

@Aurorablade
Copy link
Contributor Author

I am adding in a blood pustile ability kinda copying some things from a blob code.
Things i could use a gentle slap to the head on:
Should i be using another cooldown var instead of shareing with gorethrow?

Its supposed to throw out gibs and blood about every 60 secs, I got it working i think but dunno if its a good method.

Was going to add attack hand code...MEssed it up...>.>

i would LOVE to use the BadFeeling artifact messages but i am unsure how to call that proper.

I was not sure where to put my blob core recolor for the node so i used blood.dmi
YES I KNOW the sprite is a silly recolor..but thats my thing, i recolor/edit sprites..don't ask me to do one from scratch.

I think i am getting a cold so forgive cringe worthy code errors.

blood pustile..ewwwww
@Aurorablade Aurorablade changed the title [DNM] TG Slaughter Demon Port Remake, THE REGIBBINGING [DNM][Need Feedback] TG Slaughter Demon Port Remake, THE REGIBBINGING Jun 28, 2015
@Aurorablade
Copy link
Contributor Author

@Fox-McCloud I was doing a compile to test out my paranoia changes and got the following error, i have not touched these files so i am unsure why its doing it
loading paradise.dme
code\modules\garbage collection\garbage_collector.dm:108:error: proc definition not allowed inside another proc
code\modules\garbage collection\garbage_collector.dm:144:error: unbalanced }
paradise.dmb - 2 errors, 0 warnings

@Aurorablade
Copy link
Contributor Author

OH HI TRAVIS HOW ARE YOU

@Aurorablade
Copy link
Contributor Author

Note i am going to fix the travis errors but i want to get that previous mentioned GC error out of the way so i can ACTUALLY COMPILE AND TEST

@Aurorablade
Copy link
Contributor Author

OKAY fixed my issue, Apprently i had to update Byond to the BETA version makes the more you know rainbow

Weee! paranoia works!
@Aurorablade
Copy link
Contributor Author

I am actually pretty happy with where this is unless someone sees a code change in need to make or feels i need make it a random event proper.

The bloodnode code might need some tweaks though..

@TheDZD TheDZD removed the Do Not Merge This PR must not be merged or closed label Aug 6, 2015
@TheDZD
Copy link
Contributor

TheDZD commented Aug 6, 2015

Looked over this, looks fine. Awaiting opinion from @Fox-McCloud.

var/checking = 0


/obj/item/weapon/antag_spawner/slaughter_demon/attack_self(mob/user as mob)
Copy link
Member

Choose a reason for hiding this comment

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

not really sure this should be handled this way--this is a wizard item, so it may be best to handle it just like the revenant does; pull someone and shove them into the slaughter demon; also means you dont' ahve to have the get_candidate_answer stuff.

@ghost
Copy link

ghost commented Aug 6, 2015

Surely one day this will be merged

icon_state = "innards"

//Gib helperfunc
/mob/living/proc/GibEat(var/napped)
Copy link
Member

Choose a reason for hiding this comment

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

I have concerns about this; one of the limits on a slaughter demon's power is that he's dependent on existing blood...and this pretty much means that he self-creates blood.

@Fox-McCloud
Copy link
Member

Aside from the things I addressed...I have some concerns with the custom abilities you added. All of them generate a huge amount of blood....which is the "resource" the slaughter demon depends on....slaughter demon already saounds like it's pretty insanely strong, what with its 200 health and 30 melee damage...I don't think killing people or creating blood will be that difficult for it, to be honest.

@ghost
Copy link

ghost commented Aug 6, 2015

It actually creates way too much blood as it is simply by dragging bodies around behind it.

@Fox-McCloud
Copy link
Member

@KorPhaeron

Exactly, which is why, IMO, it doesn't need any of the spells here.

@@ -0,0 +1,117 @@
//yes i did stealfrom blobcode...
/obj/effect/bloodnode
name = "blood pustilue"
Copy link

Choose a reason for hiding this comment

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

Pustule?

@Aurorablade
Copy link
Contributor Author

Commiting changes in a moment..not sure if i need a better player mind check on the antag spawner..Much Sleepy, Many bed, wow falls asleep on keybored

Revert to the orignal TG method of just shoving a player into the daemon
on vial break.

did the same for the event (i still like using the revenant spawns
rather then carp spawns for that, not as disoriniting to me)

Added reference to BLOODCRAWL_EAT i missed

Devouring now heals more types of damage

Removes some gore abilitys...I got overzealous in wanting
blood....Keeping the spooky demon head whisper...
No don't ask why i called it Pustle...My brain frightens me...
@Fox-McCloud
Copy link
Member

Whisper was a good inclusion; I never had a problem with that one.

@Fox-McCloud
Copy link
Member

Probably pretty close to being mergable; requesting someone else review this; potentially @TheDZD or @tigercat2000

@@ -199,5 +211,5 @@

/obj/item/weapon/scrying/attack_self(mob/user as mob)
user << "\blue You can see...everything!"
visible_message("\red <B>[usr] stares into [src], their eyes glazing over.</B>")
visible_message("\red <B>[user] stares into [src], their eyes glazing over.</B>")
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be best to replace the text macro and bold with <span class='danger'>message</span>. Just minor nitpicking though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah i was gonna fix all of them in the file but i saw so many that i think my head exploded...i just changed that one user..then looked all the preexisting stuff on the file....

@Aurorablade
Copy link
Contributor Author

@Fox-McCloud @TheDZD anything else i need to fix/change?

TheDZD added a commit that referenced this pull request Aug 7, 2015
[Need Feedback] TG Slaughter Demon Port Remake, THE REGIBBINGING
@TheDZD TheDZD merged commit 171c0d2 into ParadiseSS13:master Aug 7, 2015
@Regen1
Copy link
Contributor

Regen1 commented Aug 7, 2015

!HAPPENING!

@ghost
Copy link

ghost commented Aug 7, 2015

Oh wow finally. Steal stands next.

@Aurorablade
Copy link
Contributor Author

@KorPhaeron you mean your guardian spirit thing?

@ghost
Copy link

ghost commented Aug 8, 2015

Yeah. Also lemme know how demons work out

@Aurorablade Aurorablade deleted the STILLALOTOFBLOOD branch February 13, 2016 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This PR is a new addition to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants