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

Space mines, Syndicate stealth attacks #2693

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

Conversation

Bokkiewokkie
Copy link
Contributor

About The Pull Request

This PR adds space mines, a new weapon invented by combining Syndicate cloaking technology with high-yield explosives. Currently Aspala-class submarines, Mako flak frigates, and combat carriers will be able to drop them when fleeing or patrolling.

Why It's Good For The Game

More fun stuff to die to, it also makes Syndicate systems more dangerous by spawning a few mines in them at roundstart.

Testing Photographs and Procedure

I tested it. Only shooting doesn't work nicely (you need to actually click on a mine to hit it right now)

Changelog

🆑
add: Added space mines
/:cl:

I also moved all of the torps to the left by one pixel
Watch out when flying into Syndicate space!
@Bokkiewokkie Bokkiewokkie added Feature Overmap This issue or PR is related to overmap interactions Starsystem Issues and pull requests related to star systems. labels Jul 5, 2024
Copy link
Contributor

@DeltaFire15 DeltaFire15 left a comment

Choose a reason for hiding this comment

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

Quick preliminary review.

@@ -897,6 +899,20 @@ Returns a faction datum by its name (case insensitive!)
enemy_type = pick(SSstar_system.enemy_types) //Spawn a random set of enemies.
SSstar_system.spawn_ship(enemy_type, src)

/datum/star_system/proc/spawn_mines(faction, amount)
if(!amount)
amount = difficulty_budget*2
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably short-circuit out if amount ends up as 0. Not that it causes any issues at the moment

nsv13/code/controllers/subsystem/starsystem.dm Outdated Show resolved Hide resolved
if(occupying_z)
z_level = occupying_z
for(var/i = 0, i < amount, i++)
var/obj/structure/space_mine/M = new /obj/structure/space_mine(get_turf(locate(rand(5, world.maxx - 5), rand(5, world.maxy - 5), z_level)), faction, src) //random location in the system
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably create them in nullspace or at a fixed location on the CC level instead of a random on it. As unlikely as it is for them to spawn on an admin doing fighter testing or something, it's not entirely impossible.

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 is the same way we generate anomalies, but I can definitely change it for both to be better

nsv13/code/modules/overmap/ai-skynet.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Outdated Show resolved Hide resolved
nsv13/code/modules/projectiles/projectile.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/ai_interiors.dm Outdated Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Show resolved Hide resolved
nsv13/code/modules/overmap/weapons/mines.dm Outdated Show resolved Hide resolved
Bokkiewokkie and others added 2 commits July 8, 2024 21:07
Co-authored-by: DeltaFire <46569814+DeltaFire15@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Active Test Merge Feature Overmap This issue or PR is related to overmap interactions Starsystem Issues and pull requests related to star systems.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants