If there is new stuff then probably a new ticket for that (link it to this one tho)

Just double checking, with "stuff" you also mean bugs, feedback and or answering questions you posted in your latest update?

@kju-PvPscene minPlayerDistance = 50; That means if you are closer than 50 m to the corpse or wreck it wont be removed. Please try with minPlayerDistance = 0; and report back

Actually no need, Explosion has projectile on which you can use getShotParents.

Revision: 151391

Thank you, works like a charm!

ticket?

In T178958#2565016, @BIS_fnc_KK wrote:

Make a ticket, if it is feasible in context of the event then why not. Keep in mind HitExplosion =/= Explosion

ENTITY Explosion is called alongside ENTITY HitPart has projectile entity and source in it, might as well have instigator there.

Make a ticket, if it is feasible in context of the event then why not. Keep in mind HitExplosion =/= Explosion

In T178958#2564970, @BIS_fnc_KK wrote:

Also added shotInstigator param to events:

ammo HitPart
ammo AmmoHit
entity HitPart
ammo HitExplosion
0nFlare.sqs

Also added shotInstigator param to events:

Revision: 151389

Repro hint output:

Repro RPT example:
Server:

22:20:45 #############################################################
22:20:45 # ProjectileCreated: 10: rocket_rpg7.p3d (R_PG7_F)
22:20:45 local = true
22:20:45 source = B Alpha 1-1:1 (Sa-Matra) (B_RangeMaster_F)
22:20:45 instig = B Alpha 1-1:1 (Sa-Matra) (B_RangeMaster_F)

Client:

22:20:45 #############################################################
22:20:45 # ProjectileCreated: 10: rocket_rpg7.p3d (R_PG7_F)
22:20:45 local = false
22:20:45 source = B Alpha 1-1:1 (Sa-Matra) REMOTE (B_RangeMaster_F)
22:20:45 instig = <NULL-object> ()

should be fixed

I'm surprised it event works anyhow on dead unit. I wont touch it in fear it might break something else, so just kill unit after mission start rather than starting with health 0. Gonna close this as fixed

With an eye on #constrList constituency, might also default member #type #create functions with a sensible default along similar lines. i.e.

_self getOrDefault ["#create", {_this}];

Again, could be #flag opted in either way, assuming early adoption, breaking changes, etc.

lastEntityCausingDamage is the engine pullup, so adding instigator would require the whole new implementation, not likely happening any time soon

Could also facilitate via #flags for that matter, if it is considered an early adoption breaking change.

Is this related to lack of vehicle or instigator on the entity in lastEntityCausingDamage @ T178711? I think it would be correct for entities to store both last damage vehicle and instigator and then use it for proper Killed event handler arguments and these explosions parents.

Revision: 151379

As designed, the explosion doesnt have separate instigator as the shotShell has

Revision: 151378

Could even optimize the core of the approach a little bit further.

bit of insight conveyed to me, apparently has to do with the #create mechanism and #constrList being formed. which is a bit tricky when the ctor arguments are a bit differently shaped from a base class type to derived one. the workaround is tricky, selecting around #create when specifying a #base and doing the ctor chaining manually, which seems to resolve the issue.

Thank you that you took the time to answer my inquery.

In T170647#2563304, @michaelwplde wrote:

In T170647#2563281, @michaelwplde wrote:

Probably been covered before, having had a couple of learning moments doing appropriate method, ctor, dtor chaining. #base. prefixing works for academic use cases, IAnimal to ICat or IDog, for instance.

But if your hierarchy is deeper than a couple of levels, then I think perhaps prefixing to the interface is probably best. Say, IAnimal to IDog to IAustralianCattleDog. For instance, IDie (loosely, [_sides]) to IHeroSystemBody ([_sides, _zeroes, _twos]) to IHeroSystemBodyRoll ([_sides, _times, _summary, _zeroes, _twos]), attributes loosely illustrated. Plus "Roll" method behavior comprehension, i.e. "IDie.#create", "IDie.Roll", "IHeroSystemBody.#create", and so on

Such chaining may be set during "#create" ctor, or perhaps could be part of the formal type decl itself, in order to ensure appropriate declarations are being made. Would need to tinker with that what works, seems best, etc.

Starting to notice what others have reported concerning spurious, possible scheduled unscheduled, #create callbacks. Put some systemChat reports into my #create callbacks to examine the method chaining, and I am definitely seeing multiple "default" callbacks going on.

As observed, for instance with reports such as, meaning, should SHOULD report what I passed during createHashMapObject, correct? Ultimately... I see one such report on up the chain. Then there is a second pairing involving a _this "default" what looks like default arguments.

systemChat format ["[fn_hmo_onCreateHeroSystemBodyRoll] [_this]: %1", str [_this]];

That buggerboo is going to need addressing before HMO could be considered "ready" for prime time IMO, FWIW.

The pattern even looks a bit combinatorial, IOW (in other words), as you hike the callbacks on up the method chain, that introduces another combination to the resolution.

Definitely not what I would expect be happening creating a single instance HMO.

All that was viable has been done for 2.18.
If there is new stuff then probably a new ticket for that (link it to this one tho)

Probably not 2.16
2.16 profiling branch though, shall be good enough

In T170647#2563281, @michaelwplde wrote:

Probably been covered before, having had a couple of learning moments doing appropriate method, ctor, dtor chaining. #base. prefixing works for academic use cases, IAnimal to ICat or IDog, for instance.

But if your hierarchy is deeper than a couple of levels, then I think perhaps prefixing to the interface is probably best. Say, IAnimal to IDog to IAustralianCattleDog. For instance, IDie (loosely, [_sides]) to IHeroSystemBody ([_sides, _zeroes, _twos]) to IHeroSystemBodyRoll ([_sides, _times, _summary, _zeroes, _twos]), attributes loosely illustrated. Plus "Roll" method behavior comprehension, i.e. "IDie.#create", "IDie.Roll", "IHeroSystemBody.#create", and so on

Such chaining may be set during "#create" ctor, or perhaps could be part of the formal type decl itself, in order to ensure appropriate declarations are being made. Would need to tinker with that what works, seems best, etc.

Maybe something that can be looked at

In T178888#2563300, @BIS_fnc_KK wrote:

What about https://community.bistudio.com/wiki/uniqueUnitItems is this faster?

What about https://community.bistudio.com/wiki/uniqueUnitItems is this faster?

Probably been covered before, having had a couple of learning moments doing appropriate method, ctor, dtor chaining. #base. prefixing works for academic use cases, IAnimal to ICat or IDog, for instance.

In T170647#2563059, @michaelwplde wrote:

Have my eyes on the XPS OOP effort going on, aimed at leveraging the HMO work. I understand couple of improvements, corrections, in the dev queue around that effort. Meanwhile, trying to make a somewhat uninformed usage involving some forms of method, ctor chaining. I wonder if the internals are prohibitiing some key liberties I am taking in order to achieve that.

Have my eyes on the XPS OOP effort going on, aimed at leveraging the HMO work. I understand couple of improvements, corrections, in the dev queue around that effort. Meanwhile, trying to make a somewhat uninformed usage involving some forms of method, ctor chaining. I wonder if the internals are prohibitiing some key liberties I am taking in order to achieve that.

TL;DR will review some of the comments eventually... however, can I suggest a more elaborate page detailing just _what is a HashMapObject_, particularly nuances along the lines of #Flags, what they are, what can they be, "sealed" for starters. thank you...

Is this issue back? I've observed the same behaviour since two weeks ago.

On a different note:
setMagazineTurretAmmo is broken when there are 2 players in a vehicle, it does not like split locality.

Revision: 151373

Storing stringtable keys, like "Is this defined" would be acceptable in this specific case. And also the package/container names would be okey in this specific case.
But this only applies to base game's stringtables from un-encrypted PBO's. For CDLCs I would say no.
I assume your mostly vanilla mission is unlikely to use CDLC stringtable keys anyway.

It would be very nice to see the following be implemented: