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

Ice and Sandstone walls dont spawn girders #2176

Merged
merged 4 commits into from
Oct 16, 2024

Conversation

MisterMecky
Copy link
Contributor

About the PR

Changes ice and sandstone walls to no longer spawn a girder on their destruction

Why / Balance

2024-10-09_11 04 47

How to test

  1. Spawn Ice Wall / Sandstone Wall
  2. Destroy it

Media

2024-10-09.11-13-28.mp4

Requirements

Breaking changes

Changelog

🆑

  • tweak: Ice and sandstone walls no longer spawn a girder upon their destruction

@dvir001
Copy link
Contributor

dvir001 commented Oct 9, 2024

Any other asteroid walls have the same issue or just this 2?

@whatston3
Copy link
Contributor

whatston3 commented Oct 9, 2024

Why not put this in AsteroidWallNF? Then all walls can share the same damage and spawn behaviour (none):

  - type: Destructible
    thresholds:
    - trigger:
        !type:DamageTrigger
        damage: 700 # Approximate strength as mining walls
      behaviors:
      - !type:DoActsBehavior
        acts: [ "Destruction" ]

Suggested changes up here, cherry-pick if you'd like: https://github.com/MisterMecky/frontier-station-14/compare/roidwalls...whatston3:frontier-station-14:roidwalls-suggestion?expand=1

@MisterMecky
Copy link
Contributor Author

Any other asteroid walls have the same issue or just this 2?

Only these two spawn girders from what I've found, most others are cobble walls ( no girder)

Why not put this in AsteroidWallNF? Then all walls can share the same damage and spawn behaviour (none):

  - type: Destructible
    thresholds:
    - trigger:
        !type:DamageTrigger
        damage: 700 # Approximate strength as mining walls
      behaviors:
      - !type:DoActsBehavior
        acts: [ "Destruction" ]

Suggested changes up here, cherry-pick if you'd like: https://github.com/MisterMecky/frontier-station-14/compare/roidwalls...whatston3:frontier-station-14:roidwalls-suggestion?expand=1

This is a much better solution, though 700 damage seems too high for the walls. At the moment, most walls have a strength of 100 ( all the cobble variants), sand and ice walls have a strength of 300, and necropolis walls have a whopping 1000.

Maybe 300 is a good middle point?

Copy link
Contributor

@whatston3 whatston3 left a comment

Choose a reason for hiding this comment

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

Made the damage 1000. Pending a mining redesign that gives the PTK a niche to fill beyond blasting everything in sight and sitting comfortably indoors, this is fine.

@whatston3 whatston3 merged commit c779935 into new-frontiers-14:master Oct 16, 2024
12 checks passed
FrontierATC added a commit that referenced this pull request Oct 16, 2024
@MisterMecky MisterMecky deleted the roidwalls branch October 19, 2024 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants