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

Add LocalRandom to World #15788

Merged
merged 2 commits into from Nov 11, 2018

Conversation

Projects
None yet
3 participants
@reaperrr
Copy link
Contributor

reaperrr commented Nov 4, 2018

To replace Game.CosmeticRandom later down the road.

Only ported CreateEffectWarhead for now to have something to test, but will be used by AI bot modules as well.

Suggested testcase: Add

	Warhead@3Eff: CreateEffect
		Explosions: flak_explosion_ground, small_explosion, med_explosion, artillery_explosion

to RA's 90mm weapon (ballistics.yaml) and watch allied medium tanks on shellmap shoot.

reaperrr added some commits Nov 4, 2018

Add LocalRandom to World
To - in the long term - reduce or remove Game.CosmeticRandom.

@reaperrr reaperrr force-pushed the reaperrr:LocalRandom branch from 809f491 to 85fab74 Nov 9, 2018

@pchote

pchote approved these changes Nov 10, 2018

@pchote pchote added the PR: Needs +2 label Nov 10, 2018

@obrakmann
Copy link
Contributor

obrakmann left a comment

👍

@obrakmann obrakmann merged commit cd82382 into OpenRA:bleed Nov 11, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@obrakmann

This comment has been minimized.

Copy link
Contributor

obrakmann commented Nov 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment