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

Container spawn randomisation #25

Merged
merged 2 commits into from
Jul 15, 2023
Merged

Container spawn randomisation #25

merged 2 commits into from
Jul 15, 2023

Conversation

NeutraleNull
Copy link
Contributor

Attempts to solve the issue with obvious container positions by randomise the container positions a bit.
My thought is a good empty position is better to allow blending in with the terrain.
If no position is found by the BIS function the default provided position is used.

Further thoughts:

  • We could still attempt to try a random direction and distance randomisation because any terrainobjects would be erased, keeping special care about cities and buildings
  • Maybe give the random values a CBA setting, eventough i don't think that would be to important here.

Closes #23

We attempt to find an empty spot to blend in. If nothing is found we default to the original provided position.
@NeutraleNull NeutraleNull added the enhancement New feature or request label Jul 15, 2023
couldn't get _roadBlacklist it to work and feature is already an improvement without it
@b-mayr-1984
Copy link
Member

Nice job @NeutraleNull 🙂
I made some minor changes to get it to work. Seems to work fine now.
image

I once saw this glitch, but I think even with this sometimes happening the game mode is better than before.
image image

@b-mayr-1984 b-mayr-1984 merged commit 3a56511 into main Jul 15, 2023
@NeutraleNull
Copy link
Contributor Author

watch out you removed this https://github.com/gruppe-adler/Shoot_and_Scoot.Tanoa/blob/3a56511d7e19fde1f4034a0f9e73051318cbb6fa/scripts/spot-randomizer/functions/fn_placeSupplyStation.sqf#L6C62-L6C76 without replacing the removed variable. You can replace _roadBlacklist with [].

@b-mayr-1984
Copy link
Member

b-mayr-1984 commented Jul 15, 2023

watch out you removed this https://github.com/gruppe-adler/Shoot_and_Scoot.Tanoa/blob/3a56511d7e19fde1f4034a0f9e73051318cbb6fa/scripts/spot-randomizer/functions/fn_placeSupplyStation.sqf#L6C62-L6C76 without replacing the removed variable. You can replace _roadBlacklist with [].

Hm I wonder if this got lost during merge or me just fiddeling around too much. 😕
Anyways... thx for pointing it out. I'll fix it ASAP.

b-mayr-1984 added a commit that referenced this pull request Jul 15, 2023
as discussed in #25

Co-Authored-By: Dragon <[email protected]>
@b-mayr-1984 b-mayr-1984 deleted the position_randomisation branch August 6, 2023 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fuzzy container spawn
2 participants