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

Random uniform #311

Merged
merged 7 commits into from
Jan 27, 2024
Merged

Random uniform #311

merged 7 commits into from
Jan 27, 2024

Conversation

lawhead
Copy link
Collaborator

@lawhead lawhead commented Jan 12, 2024

Overview

Updated Copy Phrase task machinery so that when using the Uniform language model, the first Inquiry of symbols are selected randomly, rather than taking the first n letters of the alphabet.

Ticket

https://www.pivotaltracker.com/story/show/186571653

Contributions

  • Updated the NBestStimuliAgent used in Copy Phrase to randomize the order of inputs prior to sorting by descending probability. This ensures that symbols with the same probability values are ordered randomly.
  • Updated the default value of the lm_backspace_prob parameter so by default it is given the same initial probability as any other symbol.
  • Bug fix

Test

  • Added unit tests for Stimuli agents
  • Ran all linting and unit tests
  • Ran a Copy Phrase task and inspected the session.xlsx output to determine that the symbols were initially ordered randomly, followed by the highest likely symbols.

@lawhead lawhead requested a review from tab-cmd January 12, 2024 00:24
@lawhead lawhead merged commit 0e111d2 into 2.0.0rc4 Jan 27, 2024
6 checks passed
@lawhead lawhead deleted the random-uniform branch January 27, 2024 01:01
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.

2 participants