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.

Sign up for GitHub

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

Jump to bottom

#169822844 ; language model histogram #91

Merged
merged 5 commits into from
Jan 29, 2020
Merged

#169822844 ; language model histogram #91

merged 5 commits into from
Jan 29, 2020

Conversation

lawhead
Copy link
Collaborator

@lawhead lawhead commented Dec 13, 2019

Overview

Tooling to output the language model probabilities while the participant is typing to allow clinicians to determine the difficulty of spelling the current letter.

Ticket

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

Contributions

  • Utility method for generating a histogram from the normalized language model output
  • Code within the copy phrase wrapper to output the histogram for each new epoch.

Test

  • python bcipy/helpers/tests/test_language_model.py
  • Integration: run a Copy Phrase task with the Language Model enabled. For every new epoch, the language model probabilities should output to the console along with a textual histogram.

lawhead and others added 5 commits October 8, 2019 16:42
@lawhead
#168649473 ; resolves Copy Phrase error when Language Model includes …
51a46b5 
…a value that exceeds the decision threshold
@lawhead
Merge pull request #87 from BciPy/non-subscriptable_bug
c9f7a3f 
#168649473 ; Copy Phrase error
@tab-cmd
pull request template
6f3aaa8
@lawhead
#169822844 ; tooling to output the language model probabilities while…
c100c88 
… the participant is typing to allow clinicians to determine the difficulty of spelling the current letter
@lawhead
Added documentation
ee95fec
@lawhead lawhead requested review from shiranD and tab-cmd December 13, 2019 22:52
@tab-cmd tab-cmd changed the base branch from master to patch December 13, 2019 22:54
@lawhead lawhead merged commit f9fed31 into patch Jan 29, 2020
@lawhead lawhead deleted the lm_visualization branch January 29, 2020 18:01
@tab-cmd tab-cmd mentioned this pull request Jan 20, 2021
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tab-cmd tab-cmd tab-cmd approved these changes

@shiranD shiranD Awaiting requested review from shiranD

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lawhead @tab-cmd