ACE3 icon indicating copy to clipboard operation
ACE3 copied to clipboard

Medical Treatment - Suture

Open BrettMayson opened this issue 3 years ago • 16 comments

When merged this pull request will:

  • [x] Add logic for sutures
  • [x] A model and ui picture

Intended as an alternative to #8424

I've have yet to make a model for Arma 3, not sure about the process. If anyone else wants to make one it would be greatly appreciated, otherwise it may be a bit before I get around to figuring it out

BrettMayson avatar Aug 31 '22 20:08 BrettMayson

For a sutures model, just a small package with one of them? Capture

and/or a box that expands into packages when one gets used from it? 15824ET-3967902197

Walthzer avatar Aug 31 '22 22:08 Walthzer

The first image there is what I tried to make by retexturing a vanilla object before learning about p3ds that can't be retextured..

BrettMayson avatar Aug 31 '22 22:08 BrettMayson

Textures are a little iffy but how about these? result

Walthzer avatar Sep 01 '22 21:09 Walthzer

Looks good to me, I'd pick a less aggressive color though

BrettMayson avatar Sep 01 '22 21:09 BrettMayson

Would be good to colour match the green from the image above

kymckay avatar Sep 01 '22 21:09 kymckay

Does look better yeah, good call. Trying to figure out a good UI picture now. result

Walthzer avatar Sep 01 '22 21:09 Walthzer

If I just take the model, this is the best it would be. Can't get legible text within a reasonable resolution. Otherwise I'd say we display it in the UI as thread with a needle? result

Walthzer avatar Sep 01 '22 21:09 Walthzer

I'd use a thread and needle yeah

BrettMayson avatar Sep 02 '22 00:09 BrettMayson

Tried some different designs and this is what looks the "best", fitting a suture into the UI is awkward :P. Currently think the "Suture" picture is the better one.

ui_pictures

Walthzer avatar Sep 03 '22 15:09 Walthzer

Can't seems to create a PR onto your fork Brett. Probably doing something wrong. How would I go about "appending" to this PR from my end?

Walthzer avatar Sep 04 '22 02:09 Walthzer

@Walthzer https://github.com/BrettMayson/ACE3/compare/consume_stitches...Walthzer:ACE3:consume_stitches post a link to your pr back here if you make one because it won't show up on this repo

PabstMirror avatar Sep 04 '22 02:09 PabstMirror

Wierd, the GitHub Interface didn't allow me to select Brett from the dropdown, huh. But that link did the trick Pabst. Created a PR for @BrettMayson to add the models to the branch: the PR

Walthzer avatar Sep 04 '22 03:09 Walthzer

Anyone know what the issue here is?

BrettMayson avatar Sep 06 '22 16:09 BrettMayson

Anyone know what the issue here is?

Sadly I do, I included the wrong models. They are trying to load textures files from the wrong path. Ugh. Will open a fix in a few.

Walthzer avatar Sep 06 '22 17:09 Walthzer

@BrettMayson Opened a PR that should fix the paths.

Walthzer avatar Sep 07 '22 16:09 Walthzer

Used this in an MP mission tonight, worked great and models are nice. Thanks @Walthzer for the contribution!

BrettMayson avatar Sep 14 '22 04:09 BrettMayson

Probably going to want to reduce the mass of the surgical kit, forgot to mention on my last post that we set it to 10. Seems to be good so far, but we could go slightly more.

BrettMayson avatar Oct 10 '22 08:10 BrettMayson

~~We are getting Cannot load texture z\wfar\addons\axe\data\suture_nohq.paa~~

@Walthzer

BrettMayson avatar Oct 22 '22 01:10 BrettMayson

That issue was due to the wrong model being copied over. Should have already been fixed by this commit. Where is the error from?

Walthzer avatar Oct 22 '22 06:10 Walthzer

Using it for several months now with no issues found.

BrettMayson avatar Feb 01 '23 11:02 BrettMayson