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

Gamma ERT Changes #9631

Merged
merged 25 commits into from Oct 9, 2018
Merged

Conversation

Birdtalon
Copy link
Contributor

@Birdtalon Birdtalon commented Sep 28, 2018

This PR introduces some sweeping changes to the Gamma ERT, as well as a couple of new features for ERT in general.

Ideas and balance decisions: LPSpartan (@Spartan6)
Contributed sprites: Shazbot. (@shazbot194)
Memager (ERT suits before tweaks from Shazbot)

Backend Changes:

ERT files moved to their own directory. Outfit datums moved to their own file.

Alters /turf/simulated/wall/proc/thermitemelt() to allow for a time to be passed to it.
Allows bodyanalyzer to be charged in rechargers.
Some variables changed to TRUE, FALSE.
Fixes some missing spans and spelling errors in ert.dm (formerly response_team.dm)

Features

Multi Lens Immolator Cannon
2018-09-03_18-12-32

Has two firing modes. "Precise" and "scatter". Precise deals 45 damage and ignites the target. Scatter fires 6 projectiles which deal 8 damage per projectile and ignites.

Portable Body Analyzer
body_scanner_print

Holds the same functionality of the body scanner machine, however requires a 10 second scan of of a mob. It then prints the same print out. Comes in an advanced and standard version which vary in cell capacity,

T4 Thermite charge
2018-09-20_13-00-01

breach_charge

Wall breaching charge similar to X4. Thermites through adjacent walls at high speed, followed by a small explosion and smoke on the other side of the wall on which it was placed.

Minor

PDW-9
unknown

Reskin of the blueshield gun to use as an ERT sidearm. Functionally identical to the blueshield's advanced stun gun.

Adds a fireproof variant of the ERT armour for gamma ERT. New sprites for these suits are below.

unknown

Amber Paranormal ERT armour with the crusader armour sprites.

Loadout changes:

  • Removed all pulse weapons.
  • Introduces PDW-9 as a side arm for red commander, gamma commander, red security, gamma engineer, gamma medic.
  • Introduces various implants to red and gamma ERT.
  • Removes sec huds from gamma ERT in exchange for implants + regular sunglasses.
  • Gamma ERT medic and engineers get implants related to their roles.
  • Gamma ERT security get multi lens immolator gun.
  • Red and Gamma ERT get portable body analyzers.
  • Security belt for Gamma ERT slightly changed.
  • Amber paranormal ERT gets a space suit with crusader armour sprites.
  • Gamma ERT get fireproof variants of their space suits.
  • ERT no longer get the standard Box and instead get a survival box only.
  • Salycilic acid pill removed from survival box in favour of autoinjector.

This started a while ago so I may have missed some things in this description. I will be going over this again at the weekend to double check my changes. However please point out any errors/missing parts.

If anyone wants to mess around with these changes and can't/don't want to compile then @ me on discord @Birdtalon#9841 and I can add your ckey to my server with admin rights and you can play around.

--
Authorised by Tigercat and Crazylemon
Merged feature Freeze fixes:
#9593, #9591, #9597

🆑 Birdtalon , LPSpartan, Shazbot, Memager
add: Portable Body Analyzer for ERT
add: Muilti Lens Immolation gun for gamma ERT
add: PDW-9 sidearm for ERT
add: T4 Thermite Charge for Gamma ERT security
add: Sprites for gamma ERT suits
tweak: Red ERT loadouts
tweak: Gamma ERT loadouts
/🆑

@tigercat2000 tigercat2000 added Feature This PR is a new addition to the game Balance This PR will modify how effective something is or isnt labels Sep 28, 2018
pda = /obj/item/pda/heads/ert
id = /obj/item/card/id/ert
l_ear = /obj/item/radio/headset/ert/alt
box = /obj/item/storage/box/responseteam
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was changed from regular box.

/obj/item/storage/box/responseteam/
name = "boxed survival kit"

/obj/item/storage/box/responseteam/New()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Equipment provided in this box was changed. Salicylic acid pill removed. Autoinjector added.


/obj/item/organ/internal/cyberimp/arm/advmop
name = "advanced mob implant"
Copy link
Contributor

Choose a reason for hiding this comment

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

advanced mob implant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👀

@Taaketa
Copy link

Taaketa commented Sep 29, 2018

Does this mean the ERTs lose the laser rifle?

@th6795
Copy link

th6795 commented Sep 29, 2018

@Taaketa

Red ERT lose the accelerator cannon and Gamma ERT lose the pulse rifle.

These are both good changes though, in my opinion because the accelerator cannon was trash anyways as it did not fit into any slots or bags and the damage was dependent on how far you were from the target which is pretty situational in my opinion.

As far as Gamma, the pulse rifle was pretty OP for that level. It caused there to be a huge gap in firepower between Red and Gamma which in-turn made it more difficult to justify a higher level ERT if something more than Red was needed.

@SkeletalElite
Copy link
Contributor

Red ERT has this as a secondary
Red also has that 3 round burst laser rifle thats reloaded with magazines

@Spartan6
Copy link
Contributor

That laser carbine for Red has been a thing for ERT for over a month, this PR is not adding it.

return

if(!ready)
to_chat(user, "<span class='notice'>The scanner beeps angrily at you! It's currently recharging - [round((time_to_use - world.time) * 0.1)] seconds remaining.")
Copy link
Contributor

Choose a reason for hiding this comment

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

close your span

if(power_supply.charge >= usecharge)
mobScan(M, user)
else
to_chat(user, "<span class='notice'>The scanner beeps angrily at you! It's out of charge!")
Copy link
Contributor

Choose a reason for hiding this comment

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

close your span

printout.name = "Scan report - [M.name]"
playsound(user.loc, 'sound/goonstation/machines/printer_dotmatrix.ogg', 50, 1)
user.put_in_hands(printout)
time_to_use = world.time + 600
Copy link
Contributor

Choose a reason for hiding this comment

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

thanks darling <3

addtimer(CALLBACK(src, /obj/item/bodyanalyzer/.proc/update_icon), 20)

else if(iscorgi(M) && M.stat == DEAD)
to_chat(user, "<span class='notice'>You wonder if [M.p_they()] was a good dog. <b>[src] tells you they were the best...</b></span>") // :'(
Copy link
Contributor

Choose a reason for hiding this comment

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

😭

Copy link
Contributor

Choose a reason for hiding this comment

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

😢

var/found_disease = FALSE
for(var/thing in target.viruses)
var/datum/disease/D = thing
if(D.visibility_flags & HIDDEN_SCANNER || D.visibility_flags & HIDDEN_PANDEMIC)
Copy link
Contributor

Choose a reason for hiding this comment

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

Tiny thing, but maybe rework so you're not just checking the same var twice...

Copy link
Contributor Author

@Birdtalon Birdtalon Sep 29, 2018

Choose a reason for hiding this comment

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

I mean, it's checking that two different bits are on. But I could just do if(D.visibility_flags) as there's only two to worry about and this already checks for both.

@SkeletalElite
Copy link
Contributor

Considering the PDW-9 is a respite of the advanced stun revolver, can a seclite be attached to it and if so does that have a sprite

@shazbot194
Copy link
Contributor

They all use the same sprite, it’s more of a question if the new x y is set.

@variableundefined variableundefined added the Test Merge Candidate Something that should / will be test merged before merging. label Oct 7, 2018
@Birdtalon
Copy link
Contributor Author

Bugs addressed:

Gamma ERT sec belt inaccessable.
Suits now are named "elite"
Gamma ERT Medic gets both his arm implants
Commander gets his enforcer
Immolator gun now goes in suit storage slot

@Birdtalon
Copy link
Contributor Author

Added sprites for gamma ERT suits.

@Birdtalon Birdtalon closed this Oct 7, 2018
@Birdtalon Birdtalon reopened this Oct 7, 2018
@variableundefined
Copy link
Contributor

@Birdtalon Travis issue with enforcer lethal ammo typepath
code\modules\response_team\ert_outfits.dm:86:error: /obj/item/ammo_box/magazine/m45/enforcer45/lethal: undefined type path
code\modules\response_team\ert_outfits.dm:94:error: /obj/item/ammo_box/magazine/m45/enforcer45/lethal: undefined type path

@Birdtalon
Copy link
Contributor Author

@variableundefined typepath was changed by #9401 - I will fix asap.

@variableundefined variableundefined removed Needs Review Test Merge Candidate Something that should / will be test merged before merging. labels Oct 9, 2018
Copy link
Contributor

@variableundefined variableundefined left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -115,6 +115,13 @@
if(R.rcell.give(R.rcell.chargerate))
use_power(200)
using_power = 1

if(istype(charging, /obj/item/bodyanalyzer))
Copy link
Contributor

Choose a reason for hiding this comment

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

Cell component when

@@ -632,3 +632,253 @@ REAGENT SCANNER
if(T.cores > 1)
user.show_message("Anomalous slime core amount detected", 1)
user.show_message("Growth progress: [T.amount_grown]/10", 1)

/obj/item/bodyanalyzer
name = "handheld body analyzer"
Copy link
Contributor

Choose a reason for hiding this comment

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

😱 We really need to componentize this if this is going to be used again

@variableundefined
Copy link
Contributor

Travis take so long..

@variableundefined variableundefined merged commit c2bf480 into ParadiseSS13:master Oct 9, 2018
@Birdtalon Birdtalon deleted the ERT-changes branch October 9, 2018 17:17
@IrkallaEpsilon
Copy link
Contributor

@Birdtalon I checked the Loadouts, the Gamma Paranormal ERT is missing a Nullrod.

@Birdtalon
Copy link
Contributor Author

@IrkallaEpsilon 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance This PR will modify how effective something is or isnt 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

10 participants