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

MBLD scrambling guidelines #1155

Merged
merged 5 commits into from
Nov 10, 2023
Merged

MBLD scrambling guidelines #1155

merged 5 commits into from
Nov 10, 2023

Conversation

Rouxles
Copy link
Contributor

@Rouxles Rouxles commented Sep 13, 2023

No description provided.

@Rouxles Rouxles requested a review from a team as a code owner September 13, 2023 08:06
wca-guidelines.md Outdated Show resolved Hide resolved
CarterKoala and others added 2 commits October 4, 2023 07:40
I think you're right, but I prefer the wording in the original change (verify there are no duplicates) as your wording doesn't really clarify doing an extra check.

Co-authored-by: Nanush7 <[email protected]>
Copy link
Contributor

@CarterKoala CarterKoala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't like using check as a noun here, I think this is better.

wca-guidelines.md Outdated Show resolved Hide resolved
Co-authored-by: Carter Kucala <[email protected]>
@Rouxles
Copy link
Contributor Author

Rouxles commented Nov 9, 2023

This one will need a review from somebody

wca-guidelines.md Outdated Show resolved Hide resolved
Copy link
Member

@Nevseros Nevseros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Nevseros Nevseros merged commit 95447d5 into draft Nov 10, 2023
1 check passed
@Nevseros Nevseros deleted the mbld-check branch November 10, 2023 14:19
@@ -93,6 +93,7 @@ To be more informative, each Guideline is classified using one of the following
- 4d++) [ADDITION] It is permitted for the puzzle to change its orientation when it is moved from the scrambler to the solving station, as long as no one is attempting to influence the randomness of the orientation (see [Regulation A2e1](regulations:regulation:A2e1)).
- 4f+) [RECOMMENDATION] The WCA Delegate should generate sufficient scramble sequences for the entire competition ahead of time, including spare scramble sequences for extra attempts.
- 4f++) [REMINDER] If the WCA Delegate generates any additional scramble sequences during the competition, the scramble sequences must be saved.
- 4g+) [RECOMMENDATION] For Multi-Blind Solving, after scrambling all the cubes, the scrambler must verify there are no duplicate scrambles.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems a bit odd to list a "must regulation" as a "recommendation". Perhaps "clarification" would be better suited?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, this should not be a recommendation. It will be fixed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants