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.

Sign up for GitHub

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

Jump to bottom

New wizard spell: Forge Arcane Golem #11787

Merged
merged 14 commits into from Sep 30, 2016
Merged

New wizard spell: Forge Arcane Golem #11787

merged 14 commits into from Sep 30, 2016

Conversation

unid15
Copy link
Contributor

@unid15 unid15 commented Sep 6, 2016 •
edited

http://ss13.moe/wiki/index.php/User:Unid/Forge_Arcane_Golem - read more about spell interactions and other stuff here

triple fireball
bff

  • Adds a new spell: Forge Arcane Golem. It requires robes to cast and has a cooldown of 20 seconds (purchasing a cooldown upgrade lowers it to 1)
  • Casting the spell creates an arcane golem. It's a fragile (50 health), slow creature that's unable to attack. However, it knows all of its master's spells (except Forge Arcane Golem and Mind Transfer), and will cast them together with the wizard, at the same targets if possible. Spell upgrades aren't transferred to golems, except for controlled fireball (unintended but fine feature)
  • Only 1 golem can be summoned at once. Purchasing amount upgrades can raise it up to 3. Casting the spell when there are already enough golems summoned will instead teleport all of your golems to your current location.
  • The golem will follow the wizard around and won't cast spells on its own. If it dies, all of the wizard's spell will go on cooldown.
  • Sprites by @SkowronX

TODO LIST

  • Get somebody to sprite a cool sprite for them
  • Fix issues that prevent golem from working at all
  • Test #

@ComicIronic
Copy link
Contributor

Wizard just got stronk. Imagine the synergies with Ring of Fire and Magic Missile.

Please restrict this to non-raging.

@PJB3005 PJB3005 added the Content Adds something. Neat! label Sep 6, 2016
@SkowronX
Copy link
Contributor

SkowronX commented Sep 7, 2016 •
edited

here's your damn sprite I'm going home

animafragment base

NO. I'M MAD. FUCK OFF GOD DAMN JESUS CHRIST.

@unid15
Copy link
Contributor Author

unid15 commented Sep 7, 2016

@ComicIronic spell upgrades aren't transfered to golems, so only stationary rings of fire will be spawned. And all of golems' spells (including magic missile and lightning) can harm the wizard, so using offensive golems is kinda risky. Unless you absolutely insist on removing them from ragin mages, I'd rather leave it be and then remove it later if it turns out to be overpowered

@SkowronX thank you

@SkowronX
Copy link
Contributor

SkowronX commented Sep 7, 2016

Oh and here's the original sprite by @FuryMcFlurry originally made for clockcult WHICH DIDNT HAPPEN qq
animafragment base

@NigglyWiggly
Copy link
Contributor

That animation

Skowron, please

@SkowronX
Copy link
Contributor

SkowronX commented Sep 7, 2016 •
edited

I wonder if anyone noticed I haven't a single fuck in the past six months or so.

NEVERTHELESS. The show must go on!

@unid15 unid15 changed the title [WIP] New wizard spell: Forge Arcane Golem New wizard spell: Forge Arcane Golem Sep 8, 2016
@SkowronX
Copy link
Contributor

should probably get a replacement sprite for it. peeps are saying it's unfitting and I honestly don't really care about this server much anymore.

PJB3005
PJB3005 reviewed Sep 14, 2016
View reviewed changes
code/modules/mob/living/simple_animal/friendly/arcane_golem.dm
var/mob/master = master_spell.holder

if(istype(master))
to_chat(master, "<span class='warning'>As your golem is destroyed, your mana leaks away!</span>")
Copy link
Contributor

Choose a reason for hiding this comment

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

Mana? What?

@unid15
Copy link
Contributor Author

unid15 commented Sep 14, 2016

Mana as in "magic energy/magic powers". Less popular definition of the word but it's valid

mana
ˈmɑːnə/
noun
(in Polynesian, Melanesian, and Maori belief) an impersonal supernatural power which can be transmitted or inherited.

@PJB3005
Copy link
Contributor

PJB3005 commented Sep 14, 2016

Yes, but I have never seen mana used in SS13 so it just seems weird to use the term.

@NigglyWiggly
Copy link
Contributor

Well in baylore, I think, it's explained that they have a 'magic organ' that can produce energy with an electrical impulse or something, and NT actively wants them dead because they can harvest these organs to basically just multiply power for free.

In our deep lore I imagine it's just a thing you're trained in using magic energy or some shit

@FudgePucker
Copy link
Contributor

Wait does the firebird spell work with them?

@unid15
Copy link
Contributor Author

unid15 commented Sep 17, 2016 •
edited

Yep, each golem summons a firebird

Shame that summon carp isn't in the spellbook, it'd synergise pretty well with this

@Shadowmech88
Copy link
Contributor

Yep, each golem summons a firebird

Do they get into their firebirds when the wizard gets into his?

@unid15
Copy link
Contributor Author

unid15 commented Sep 17, 2016

No, they only mimic spellcasting, nothing else

That would be cool however

@clusterfack clusterfack merged commit 4505af9 into vgstation-coders:Bleeding-Edge Sep 30, 2016
@SkowronX
Copy link
Contributor

What no one wanted to make something to replace that abomination?

@Exxion
Copy link
Member

Exxion commented Sep 30, 2016

What you forget is that your worst sprites are still leagues better than most people's best sprites

@SkowronX
Copy link
Contributor

SkowronX commented Oct 1, 2016

Sucks to be vg.

StarSmasher44 referenced this pull request in StarSmasher44/NSVCenturion Nov 24, 2016
@unid15 @StarSmasher44
New wizard spell: Forge Arcane Golem (#11787)
b49ab2e 
* Adds arcane golem

* F

* Ah

* V2.0

* Gramer

* Compile

* What the fuck was I smoking?

* Last tweaks

* Fixes golems taking damage from ring of fire

* Oh hey I found some free time

* Fix for something

* update CL with more accurate info

* remove mistake

* mistake fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ComicIronic ComicIronic ComicIronic left review comments

@PJB3005 PJB3005 PJB3005 left review comments

Assignees
No one assigned
Labels
Content Adds something. Neat!
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
8 participants
@unid15 @ComicIronic @SkowronX @NigglyWiggly @PJB3005 @FudgePucker @Shadowmech88 @Exxion