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
init spawn places rewrite #2668
init spawn places rewrite #2668
Conversation
3ed6eed
to
382e67e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least it's filling the markers the right way up now.
{ | ||
Error_1("InitSpawnPlaces: Marker %1 is not long enough for vehicles, required are 10 meters!", _x); | ||
} | ||
else | ||
{ | ||
//Cleaning area | ||
private _radius = sqrt (_length * _length + _width * _width); | ||
private _radius = [0,0] vectorDistance [_width, _height]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could just do vectorMagnitude [_width, _height]
.
_realLength = _vehicleCount * 4; | ||
_realSpace = (_length - _realLength) / (_vehicleCount + 1); | ||
_realSpace = (_width - _realLength) / (_vehicleCount + 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What this does is increase the spacing between vehicles if there's some extra space left over in the marker. This is a bad idea. If the spacing is variable and in some cases not enough, then we're gonna have occasional spawn explosions that are difficult to pin down. Better to keep the spacing consistent and adjust until it doesn't blow things up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
* removed nasty distance * fixed wrong variables * renamed width to height, because arma is dumb * fixed height * change for bob
What type of PR is this.
What have you changed and why?
Information:
fixes the wrong variable being checked
Please specify which Issue this PR Resolves.
closes #2667
Please verify the following and ensure all checks are completed.
Is further testing or are further changes required?
How can the changes be tested?
Steps:
Notes: