Skip to content

Fix bouncing 40mm smoke#4648

Closed
BaerMitUmlaut wants to merge 1 commit into
masterfrom
fix-bouncy-smoke
Closed

Fix bouncing 40mm smoke#4648
BaerMitUmlaut wants to merge 1 commit into
masterfrom
fix-bouncy-smoke

Conversation

@BaerMitUmlaut
Copy link
Copy Markdown
Member

When merged this pull request will:

  • Fix rubber 40mm smoke shells

@BaerMitUmlaut BaerMitUmlaut added fault/BI kind/enhancement Release Notes: **IMPROVED:** labels Nov 9, 2016
@jonpas jonpas added this to the 3.8.2 milestone Nov 9, 2016
@commy2
Copy link
Copy Markdown
Contributor

commy2 commented Nov 9, 2016

please explain what's going on here.

@jonpas
Copy link
Copy Markdown
Member

jonpas commented Nov 9, 2016

40mm smokes bounce a lot. Fire it at a hill and watch it bounce up and over. This fixes it.

@PabstMirror
Copy link
Copy Markdown
Contributor

@BaerMitUmlaut
Copy link
Copy Markdown
Member Author

shotSmokeX, the simulation type for the current 40mm smoke grenades, is inherited from hand thrown smoke grenades, which are supposed to bounce. Because the 40mm ones travel at much higher speed, they bounce excessively when hitting the ground, making you miss your target every single time, which renders them useless for target marking - the main purpose of them irl.
This removes the bouncyness. I copied the shotSmoke simulation type from the artillery smoke shell.

@commy2
Copy link
Copy Markdown
Contributor

commy2 commented Nov 9, 2016

So this PR changes shotSmokeX to shotSmoke to remove physx bounces of UGL smoke grenades?
Are there any other consequences changing the simulation has? Are all sounds and effects still working? Or do we have to redefine some entries?
shotSmoke seems to be usable if artillery shells and mortar rounds use it

@PabstMirror
Copy link
Copy Markdown
Contributor

the round that comes out of arty/mortars is a a simulation = "shotDeploy";
which becomes the shotSmoke ammo on impact
doing it that way should guarantee no bounce.

We should double check how shotSmoke performs with air resistance as I believe it is disabled on some types. Making sure HE and Smoke rounds fly with similar flight paths in wind.

@commy2
Copy link
Copy Markdown
Contributor

commy2 commented Nov 9, 2016

Yeah. And I somehow don't think we should totally get rid of the bouncing. Why not modify deflecting or even add a custom deflectionSlowDown (if that works)?

deflecting = 0; should disable it without changing the simulation.

@BaerMitUmlaut
Copy link
Copy Markdown
Member Author

A 40mm smoke shouldn't bounce unless it hits concrete or something it can't penetrate at all. In Arma that's rather the exception than the norm though.

deflecting = 0; should disable it without changing the simulation.

You're right, it should. Doesn't though.

@BaerMitUmlaut
Copy link
Copy Markdown
Member Author

Bad news: sound doesnt work. Good point commy. I'll try tweaking the X one again then...

@Neviothr
Copy link
Copy Markdown
Contributor

Neviothr commented Nov 10, 2016

I used this to disable the bouncing:

class CfgAmmo {
    class SmokeShell;
    class G_40mm_Smoke : SmokeShell {
        simulation = "shotSmoke";
        deflectionSlowDown = 0.3;
    };
};

Copy link
Copy Markdown
Contributor

@commy2 commy2 left a comment

Choose a reason for hiding this comment

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

shotSmoke is not an option. read above.

@BaerMitUmlaut
Copy link
Copy Markdown
Member Author

Closing because shotSmokeX is simply not fixable on our side.

@jonpas jonpas removed this from the 3.8.2 milestone Nov 11, 2016
@jonpas jonpas unassigned bux Nov 11, 2016
@commy2 commy2 deleted the fix-bouncy-smoke branch November 11, 2016 20:46
@nicolasbadano
Copy link
Copy Markdown
Contributor

😢 It'd have been awesome

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

Labels

fault/BI kind/enhancement Release Notes: **IMPROVED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants