-
Notifications
You must be signed in to change notification settings - Fork 3
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
Sequence picking #25
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The way COSplay currently picks sequences is somewhat intricate, and does no actually support randomized picking. This doesn't necessarily mean it should be changed (other than fixing randomization support), but the behaviour should either be simplified or precisely documented:
If the
sequence_library
directory contains:sequence*.tsv
- the first file in alphabetical order is always pickedsequence*.tsv
and files namedsequence*.tsv
- the first file in alphabetical order among the files namedsequence*.tsv
is pickedsequence*.tsv
- the first file in alphabetical order among the files is pickedThis is tested based on the terminal feedback from the COSplay CLI, with a
sequence_library
directory containing among others a.known_sequences
directory repositing frequently used sequences under a<trial_ID>.tsv
format - however, the presence and absence of this directory does not impact the picking behaviour.The text was updated successfully, but these errors were encountered: