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

Cargo - Fix collision when unloading a IDAP 20m long container from HEMTT #8344

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

Vdauphin
Copy link
Contributor

@Vdauphin Vdauphin commented Jul 31, 2021

Each time you unload Land_Cargo20_IDAP_F from a HEMTT, the container collide with the vehicle and go in space
This is due to [cursorObject, a, player, 10, true] call ace_common_fnc_findUnloadPosition whitch return a too close position for this container. The calculation of that position mainly depend on this:

getNumber (configFile >> "CfgVehicles" >> "Land_Cargo20_IDAP_F" >> "ace_cargo_size")

return 4

getNumber (configFile >> "CfgVehicles" >> "Land_Cargo20_military_green_F" >> "ace_cargo_size")

return 50

As we notice there is a big difference between a Land_Cargo20_military_green_F and Land_Cargo20_IDAP_F

When merged this pull request will:

@Vdauphin Vdauphin changed the title Cargo - Fix unloading of IDAP 20m long container colliding with HEMTT Cargo - Fix collision when unloading a IDAP 20m long container from HEMTT Jul 31, 2021
@PabstMirror PabstMirror added this to the 3.14.0 milestone Aug 4, 2021
@PabstMirror PabstMirror added the kind/bug-fix Release Notes: **FIXED:** label Aug 4, 2021
@PabstMirror PabstMirror merged commit 34069ed into acemod:master Aug 4, 2021
@dedmen
Copy link
Contributor

dedmen commented Aug 18, 2021

Shouldn't you calculate radius from bounding box and use that? instead of space config entry?

@PabstMirror
Copy link
Contributor

you can load cargo via classname so we won't always have a physical object to get boundingBox

@Vdauphin Vdauphin deleted the patch-7 branch October 5, 2021 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logistics Box not working
3 participants