added gunbag for DMRs #3749

ir0n1e · 2016-05-03T08:36:28Z

Gunbag for DMRs
~~Known issues: Mass calculation of full gunbag - cause of ArmA restrictions~~
Screenshots (tan)
http://imgur.com/a/r9l36

jonpas · 2016-05-03T08:44:38Z

Replace tabs with 4 spaces.

Can you also give a broader description of the feature?

BaerMitUmlaut · 2016-05-03T08:57:29Z

Looks like this adds an interaction to add a weapon virtually to a gunbag backpack.

This won't work with weapons that have scopes by default which were then removed by a script.

Additionally I don't see the need for this at all. You can already put weapons in your backpack. Adding a larger backpack to the game would have the same effect without the potential for bugs and issues.

ir0n1e · 2016-05-03T09:06:31Z

@BaerMitUmlaut
Part1: correct
Part2: Possible but could not repuduse it in tests, weapon will be stored with all attachments
Part3: Your opinion

bux · 2016-05-03T09:14:35Z

@ir0n1e
Part 2 isn't possible in Arma 1.58 with the commands that are available.

Example: You equip the weapon srifle_DMR_01_ACO_F which has a RedDot when created (hence the ACO in the name).
You now remove the RedDot and mount a MRCO sight onto the weapon. You now aren't able to restore the weapon using the srifle_DMR_01_ACO_F class name with a MRCO sight equipped.

BaerMitUmlaut · 2016-05-03T09:22:57Z

Part 3 is not an opinion. What advantages does this have that a large backpack doesn't have? I see the following disadvantages:

breaks with some weapons because of how attachments work
removes the effect of weight of weapons in the backpack and is independent of what weapon is in the backpack (an SMG would weigh as much as an MMG)
removes the interaction of opening the backpack and switching out your weapon manually (realism)

ir0n1e · 2016-05-03T09:29:39Z

@bux578 Jabb, but I read current weapon config with EFUNC(common,getWeaponState)
and rebuilding the weapon to this state.

@BaerMitUmlaut you are right, but you can't store a G82 in any backpack (realism).
My prefered solution is a weapon in the inventory of the gunbag, but it's not possible to buildup the weapon with all attachments in backpack.

BaerMitUmlaut · 2016-05-03T09:32:21Z

This would work with making a new backpack that is simply larger.

ir0n1e · 2016-05-03T09:35:05Z

The G82 is more then 60 inches long, the largest backpack has less then 30 inches. You have to dismatle the weapon to store it in a backpack. (realism
)

BaerMitUmlaut · 2016-05-03T09:36:51Z

This would work with making a new backpack that is simply larger.

ir0n1e · 2016-05-03T09:38:20Z

Knowing that. Your comment was realism.

ir0n1e · 2016-05-03T09:41:23Z

@commy2 needing your opinion, caues it's your initial idea!

commy2 · 2016-05-03T09:43:17Z

This would work with making a new backpack that is simply larger.

Not true. "Long rifles" (anything that isn't a 5.56mm or 6.5mm by vanilla standards) cannot be put inside a bagpack, regardless of how little mass they have or how much cargo capacity the bagpack has.

    class Rifle : RifleCore {
        class WeaponSlotsInfo {
            allowedSlots[] = {901};
        };
    };

    class Rifle_Long_Base_F : Rifle_Base_F {
        class WeaponSlotsInfo : WeaponSlotsInfo {
            allowedSlots[] = {};
        };
    };

701 vest
801 uniform
901 backpack

BaerMitUmlaut · 2016-05-03T09:46:17Z

Why not change the configs accordingly then?

commy2 · 2016-05-03T09:47:48Z

breaks with some weapons because of how attachments work

I don't see how.

Store weapon info:
While it's true that pre-equipped weapons are reported as their class names with attachments, the primaryWeaponItems command still reports only the attachments currently there, including those added automatically.

Restore weapon:
You use addWeapon, removeAllPrimaryWeaponItems and then addPrimaryWeaponItem and can fully restore a primary weapon. The problem comes with launchers, because the commands are bugged for those.

Pre-quipped weapons are not a problem.

BaerMitUmlaut · 2016-05-03T09:49:30Z

That's not the case right now though.

commy2 · 2016-05-03T09:49:31Z

Why not change the configs accordingly then?

Then you could put (multiple of) those weapons inside even the smallest backpacks, even though they shouldn't have the space for it. It's just the cheap vanilla inventory system.

commy2 · 2016-05-03T09:50:11Z

That's not the case right now though.

I haven't checked the PR, but it is possible to achive (with primary weapons)

ir0n1e · 2016-05-03T09:51:12Z

Test it and you will see it works. Script stores weapon, and attachements. If you pull out the weapon all attacements will be build up.
Problem is mass of gunbag, cause we can't build up the weapon in a vehicle.

commy2 · 2016-05-03T09:54:25Z

Problem is mass of gunbag, cause we can't build up the weapon in a vehicle.

while this is true, you could still adjust the "loadCoef" via setUnitTrait to create some kind of "virtual mass". I haven't tested the command for bugs though ...

ir0n1e · 2016-05-03T09:57:04Z

Will test that, but first time read this command. Have tested with "setmass", but it's not working for player vehicle.

ir0n1e · 2016-05-03T10:04:46Z

@commy2 setUnitTrait sets unit specialisation..? Do you have time for a TS conversation today?

BaerMitUmlaut · 2016-05-03T10:05:26Z

Try adding weapon that has attachments by default for example arifle_TRG21_ARCO_pointer_F. Now drop all attachments, put it in the backpack and take it out again. I'm pretty sure this creates new attachments right now. I cannot try it right now.

ir0n1e · 2016-05-03T10:08:03Z

@BaerMitUmlaut jabb, the script will working so, but it's nothing that I can change.

commy2 · 2016-05-03T10:08:14Z

Do you have time for a TS conversation today?

Yes, 1700B

setUnitTrait sets unit specialisation..?

I think that was what they were going for, but they ended up also enabling editing some other numeric values.

https://community.bistudio.com/wiki/setUnitTrait

ir0n1e · 2016-05-03T10:09:51Z

@commy2 1700 thats a date!

BaerMitUmlaut · 2016-05-03T10:11:50Z

You need to use removeAllPrimaryWeaponItems first.

jokoho48 · 2016-05-08T20:28:24Z

but @BaerMitUmlaut what is happening if someone want to put RHS or CUP or BW weapons like this in there backpack and its dont work... they came to use and Blame us that we broke it.
i realy would like it if no other mod would break it again

BaerMitUmlaut · 2016-05-08T20:31:20Z

If they inherit from Rifle_Long_Base_F as they probably should there are no issues. If they don't and manually disable adding them to backpacks, it's clearly their fault. I don't see the issue with that.

ir0n1e · 2016-05-10T10:26:28Z

At this time script checks a config value. Like swapbarrol. The modders have to add this value in there ace_comp config. If you check "Rifle_Long_Base_F", LMGs and MMGs will fit into gunbag.

BaerMitUmlaut · 2016-05-10T10:32:12Z

What's the problem with putting an MMG in the gunbag? We even have deployable HMGs in Arma that come in backpacks. Why not MMGs now?

ir0n1e · 2016-05-10T11:03:54Z

Don't have a major problem with that, but it schould be a bag for sniperrifles in my opinion. In real you will carry a Barrett or a M24 in this bag. Or something similar.

ir0n1e · 2016-05-10T15:14:50Z

@commy2 Mass calculation: Which mass i have to pass (config mass or Kg/Pounds)?
Virtual mass is not recognized by EFUNC(movement,getWeight)!

ir0n1e · 2016-05-10T20:56:40Z

KK, think I have to pass config mass, cause of #3779

commy2 · 2016-05-10T22:44:34Z

addons/gunbag/functions/fnc_offGunbag.sqf

+removeAllPrimaryWeaponItems _unit;
+
+{
+  _unit addWeaponItem [_weapon, _x];


4 spaces indent

Everywhere actually. :)

jokoho48 · 2016-05-13T02:32:38Z

ok .. this time in the right PR... fu github why did you do this...

i just took a look at the model and found 2 thing that you should improve, the Polycount in this parts of the model is way to high
http://puu.sh/oNmPh/b985503f8b.jpg
http://puu.sh/oNmTj/ad8adb18ac.png

ir0n1e · 2016-05-13T19:34:35Z

@jokoho48 will see what I can do. But I hate bumpmapping.

jokoho48 · 2016-05-13T19:38:50Z

i think that can be dont without any normal

ir0n1e · 2016-05-18T10:10:50Z

I'm done. I Think, everything should pass ACE3 checks.

ir0n1e · 2016-05-30T09:10:18Z

Merge status?

SzwedzikPL · 2016-05-30T11:31:07Z

Gunbag carried on launcher slot would be an awesome feature

added gunbag

9567b58

thojkooi added the kind/feature Release Notes: **ADDED:** label May 3, 2016

ACE syntax

d3500d4

added missing }

a34490a

commy2 mentioned this pull request May 10, 2016

show virtual load in inventory menu #3779

Merged

ir0n1e changed the title ~~added gunbag~~ added gunbag for DMRs May 10, 2016

Changed mass to config mass - cause of acemod#3779

6c76d5b

commy2 reviewed May 10, 2016
View reviewed changes

ir0n1e added 8 commits May 17, 2016 10:57

dissolved some edges

1a14634

Merge branch 'master' of https://github.com/acemod/ACE3 into gunbag

6d98852

fixed idention tabs are 4 spaces now

c891633

adapted to acemod#3378 and added support for UGL

7ef0240

clean up

1302ccb

added readme

3e9cf73

Merge branch 'gunbag' of https://github.com/ir0n1e/ACE3 into gunbag

8f76e5a

added gunbag to ACE_Box_Misc

4a7f552

thojkooi added this to the 3.6.0 milestone Jun 11, 2016

commy2 self-assigned this Jun 11, 2016

commy2 mentioned this pull request Jun 12, 2016

Gunbag ace #3903

Merged

commy2 merged commit 4a7f552 into acemod:master Jun 12, 2016

jonpas mentioned this pull request Mar 26, 2018

Add Switching of Weapon from gunbag. #6212

Closed

added gunbag for DMRs #3749

added gunbag for DMRs #3749

Conversation

ir0n1e commented May 3, 2016 • edited Loading

jonpas commented May 3, 2016

BaerMitUmlaut commented May 3, 2016

ir0n1e commented May 3, 2016 • edited Loading

bux commented May 3, 2016 • edited Loading

BaerMitUmlaut commented May 3, 2016

ir0n1e commented May 3, 2016

BaerMitUmlaut commented May 3, 2016

ir0n1e commented May 3, 2016 • edited Loading

BaerMitUmlaut commented May 3, 2016

ir0n1e commented May 3, 2016

ir0n1e commented May 3, 2016

commy2 commented May 3, 2016

BaerMitUmlaut commented May 3, 2016

commy2 commented May 3, 2016 • edited Loading

BaerMitUmlaut commented May 3, 2016

commy2 commented May 3, 2016

commy2 commented May 3, 2016

ir0n1e commented May 3, 2016

commy2 commented May 3, 2016

ir0n1e commented May 3, 2016 • edited Loading

ir0n1e commented May 3, 2016

BaerMitUmlaut commented May 3, 2016

ir0n1e commented May 3, 2016

commy2 commented May 3, 2016 • edited Loading

ir0n1e commented May 3, 2016

BaerMitUmlaut commented May 3, 2016

jokoho48 commented May 8, 2016 • edited Loading

BaerMitUmlaut commented May 8, 2016

ir0n1e commented May 10, 2016

BaerMitUmlaut commented May 10, 2016

ir0n1e commented May 10, 2016 • edited Loading

ir0n1e commented May 10, 2016

ir0n1e commented May 10, 2016 • edited Loading

commy2 May 10, 2016

Choose a reason for hiding this comment

jonpas May 10, 2016

Choose a reason for hiding this comment

jokoho48 commented May 13, 2016

ir0n1e commented May 13, 2016

jokoho48 commented May 13, 2016

ir0n1e commented May 18, 2016

ir0n1e commented May 30, 2016

SzwedzikPL commented May 30, 2016 • edited Loading

ir0n1e commented May 3, 2016 •

edited

Loading

ir0n1e commented May 3, 2016 •

edited

Loading

bux commented May 3, 2016 •

edited

Loading

ir0n1e commented May 3, 2016 •

edited

Loading

commy2 commented May 3, 2016 •

edited

Loading

ir0n1e commented May 3, 2016 •

edited

Loading

commy2 commented May 3, 2016 •

edited

Loading

jokoho48 commented May 8, 2016 •

edited

Loading

ir0n1e commented May 10, 2016 •

edited

Loading

ir0n1e commented May 10, 2016 •

edited

Loading

SzwedzikPL commented May 30, 2016 •

edited

Loading