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

Implement new ship: Sunder #6990

Closed
wants to merge 17 commits into from
Closed

Implement new ship: Sunder #6990

wants to merge 17 commits into from

Conversation

eebop
Copy link

@eebop eebop commented Jun 29, 2022

Content

Summary

This implements the Sunder as a cheap (ish) human mining ship. The artwork was created in PR 6895

Save File

test2 test2.txt

Testing Done

The ship seems well-balenced as a light freighter with some mining capacity (i. e. strong engines). Although it has some weapons (to break the asteroids, it loses in combat to an intercepter.

Performance Impact

N/A

@eebop
Copy link
Author

eebop commented Jun 29, 2022

For some reason, every tab in ships.txt got deleted automatically? If this is a problem I can go in and add them all back

@Saugia
Copy link
Collaborator

Saugia commented Jun 29, 2022

I believe @Amazinite had mentioned he wants to implement a few new things along with the Sunder, such as mining lasers, some type of tractor turret, etc. when he gets to it. The drones in that PR were also specifically designed for the Sunder, as the model itself has a rack for 2 drones. In the image of that PR you can see it carrying two of them underneath. These too would have the new mining laser.

I believe this is a little premature addition of the ship, because of all the new implementations that are meant to come with it. Perhaps it can still be implemented like this as a basic representation before we get to the new content for it? Either way, we'll have to see what Derpy says as he specifically requested these models.

@Hecter94
Copy link
Member

Hecter94 commented Jun 29, 2022

For some reason, every tab in ships.txt got deleted automatically? If this is a problem I can go in and add them all back

What IDE are you using to make changes? Issues with tabbing are often the result of using a word processor rather than an IDE like Sublime, Eclipse or N++

Those tab issues will need to be fixed before merging can be considered as it seems to be causing a lot of merge conflicts and makes reviewing the PR difficult.
I'd suggest making a copy of just your changes, reverting the file back to vanilla and then using a program like the ones mentioned above to add your changes back in to avoid any extraneous file edits.

If you need any help, there's an active Discord where you can ask any questions you might have, if you're not already aware:
https://discord.gg/ZeuASSx

@Amazinite
Copy link
Collaborator

As @Saugia mentioned, there's a lot more needed for this ship than just the stats. It also needs the drone, various outfits that we either don't have sprites and stats for or just outright aren't possible in the game right now, and preferably the various AI changes that Sam has open PRs for.

@Amazinite Amazinite added the content A suggestion for new content that doesn't require code changes label Jun 29, 2022
@eebop
Copy link
Author

eebop commented Jun 29, 2022

Oh. I didn't know that it wasn't supposed to be made yet. I was already going to implement the mining drone and add a mining laser that I had been thinking about for a while.

@MasterOfGrey
Copy link
Member

I was already going to implement the mining drone and add a mining laser that I had been thinking about for a while.

Sounds like you might save some people some work though here. :)

@eebop
Copy link
Author

eebop commented Jul 13, 2022

All that's needed now is some kind of escorts:mine instruction and some graphics (and proof reading). One thing that this adds that I'm not so sure about is the mining laser also doubles as a very very weak shield-piercing laser so i don't know if that'll get aproved.

@eebop
Copy link
Author

eebop commented Jul 13, 2022

One important pr I'm waiting on is #6840

@Zitchas
Copy link
Member

Zitchas commented Jul 13, 2022

Given that this is a human ship, and an early game so at that, I really don't think the shield piercing is appropriate. At best that might be a post automata invasion development.

@Hurleveur
Copy link
Member

The description of the weapon itself seems too advanced for human tech level, I think.
We don't even have tractor beams in game yet, and even if we did this seems like a t2-t3 tech, manipulating gravity with a null field or whatever doesn't seem easy. Remnants could maybe develop something like too, this with their advanced scientific knowledge.

This weapon does such a negligible amount of damage that piercing or not won't matter anyway. I very much like the idea of it, but I'd imagine this to be the way Quargs/Exiles mine things.

@Amazinite Amazinite self-assigned this Aug 5, 2022
Copy link
Contributor

@warp-core warp-core left a comment

Choose a reason for hiding this comment

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

There's a lot of whitespace changes that shouldn't be here.

The Mining Drone is better in every way than the Combat Drone.
This could perhaps be an argument to buff the Combat Drone, but then I'd say the Flea needs buffing as well.
My suggestions are:

  • Remove the shields
  • Reduce the hull
  • Increase mass
  • Reduce the outfit space
  • Add some cargo space

engine 5 29
explode "tiny explosion" 15
"final explode" "final explosion small"
description "Thing."
Copy link
Contributor

Choose a reason for hiding this comment

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

This is gonna need a little more.

Copy link
Contributor

@warp-core warp-core left a comment

Choose a reason for hiding this comment

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

Sorry, I hit the wrong button before, didn't mean to submit then.
This Repulsion Laser has a number of problems:

  • Inappropriate thumbnail
  • Called a 'laser' but behaves like a blaster.
  • Doesn't have unique projectile sprite, sound, or effect (not necessarily an issue, but doesn't seem great to use Energy Blaster effects since it'll make it difficult to tell the two apart, and they are likely to appear in similar places).
  • Way too big for the drone you've mounted it on.
  • Probably shouldn't do piercing.
  • As a ship to ship weapon, it's not very performant, and narratively, the technology described in the description probably doesn't have much place in human space at this stage.

Personally, I would just remove this weapon from this PR and use a Beam Laser on the drone (until an actual mining laser using planned features is implemented).

@roadrunner56 roadrunner56 added the text - short Content that contains short blocks of text such as jobs, small missions, descriptions, etc. label Aug 8, 2022
@warp-core warp-core added the waiting on OP The OP needs to provide something, e.g, making requested changes, posting assets, etc. label Aug 13, 2022
@Amazinite
Copy link
Collaborator

As I mentioned earlier, there's a lot more that the Sunder needs than just stats. There are a number of other PRs that first need finalized and merged that add new mechanics or AI behaviors that the Sunder will take advantage of, and without those the Sunder won't really work as it's intended to. Aside from that, the proposed stats for the Sunder, mining drone, and mining laser are rather far off from what I believe they should be. So given that, I'd rather just close this and revisit the Sunder once it's actually ready to be implemented.

@Amazinite Amazinite closed this Aug 19, 2022
@Amazinite Amazinite removed their request for review August 19, 2022 17:56
@Amazinite Amazinite removed the waiting on OP The OP needs to provide something, e.g, making requested changes, posting assets, etc. label Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content A suggestion for new content that doesn't require code changes text - short Content that contains short blocks of text such as jobs, small missions, descriptions, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants