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

Improve UI for enabling experimental PRNG combat #4021

Open
slavrenyuk opened this issue Apr 12, 2019 · 1 comment

Comments

Projects
None yet
2 participants
@slavrenyuk
Copy link

commented Apr 12, 2019

Experimental PRNG combat presented in this thread https://forums.wesnoth.org/viewtopic.php?f=6&t=48160 looks very promising. However, it is hard to find that option (it is the last item in the advanced preferences) and it is not user friendly to enable it (both checkboxes "Use experimental PRNG combat" in advanced preferences and "Save random seed" in the campaign selection menu must be enabled).

The feature is currently available only for singleplayer campaigns. In order to promote it to more players I propose to:

  1. Move the "Use experimental PRNG combat" checkbox from the advanced preferences to the campaign selection menu.
  2. Once the "Use experimental PRNG combat" checkbox in the campaign selection menu is selected, "Save random seed" checkbox is also automatically selected and becomes non-editable.
  3. Rename the "Use experimental PRNG combat" to something more specific and user friendly. For example, "Use more predictable random generator for combat" or "Reduce random deviations in combat"
@slavrenyuk

This comment has been minimized.

Copy link
Author

commented Apr 12, 2019

That's probably too early to talk about that, but since I already spent some time thinking about this "Experimental PRNG combat" I would like to share my thought with developers. In my opinion this feature is too deterministic. What if the experimental RNG falls back to the default RNG in, for example, 5% of attacks? Once again - just wanted to share this thought, no need to discuss it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.