-
Notifications
You must be signed in to change notification settings - Fork 4
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
Self-voicing: Verbose and brief options for Interactive Mode #196
Comments
Extension of #193 |
@jessegreenberg, I added brief and verbose "density phrases" to the design doc for when Constant Size is checked. |
Here's my evaluation notes of our initial design & implementation. We can chop these notes into new issues as needed. Issues for 1.1.0-dev.2When set to Brief Explore mode & Brief Interactive mode
When set to Brief Explore mode & Verbose Interactive mode
When set to Verbose Explore mode & Verbose Interactive mode
When set to Verbose Explore mode & Brief Interactive mode
When either Verbose or Brief Interactive options are selected
Additional issues found by JG:
|
@jessegreenberg, I don't know how to make the bullets tidier. I did my best. We can break these issues up into separate issues as we decide to address them. |
@jessegreenberg, can you you give me an idea of when you can work on the issues in #196 (comment) think they are small. Do you need me to split them up a bit? |
Hmm, I tried to get this to happen with brief interactive mode and haven't been able to observe it in about 10 minutes of trying. Its possible this has been fixed since #196 (comment). @terracoda if you observe this on master please let me know, but checking off for now. EDIT: Same thing for
I haven't been able to get this to happen after several minutes, checking off for now. EDIT: Same thing for
Checking off for now. |
For #6 I went with
Just because it was very easy to do. Custom alerts at the end of the track are possible however, if we want to explore that more. |
@terracoda here is an updated link with items in the list addressed: This still doesn't include output from focus yet. Ill be working on that next but it doesn't seem related to this issue. As I mentioned, I wasn't able to reproduce the bugs you reported. Code around those has changed, so perhaps they were accidentally fixed. Let me know if you still encounter them! |
Here is an updated version that includes output from focus: |
@jessegreenberg, I have found a few additional issues
@jessegreenberg, on another note, @emily-phet and I had a deep conversation about next steps, which differs a lot from this version, so if any of the above are really tough issues, let's discuss before spending too much effort on a fix we might not use. We can address them in the new approach, if need be. |
This Self-Voicing design is what we are calling Paradigm 1. |
@terracoda I am curious about your thoughts on this issue at the moment. Should we do any more to improve this prototype or is this one phasing out? |
@jessegreenberg This version is an "artifact". One we want to be able to refer to, share with colleagues as a contrasting example, etc. but not move forward with for further design iterations or to move towards publication. |
@jessegreenberg, this version might be a good candidate for a build that we can save to share in papers - something we can freeze in time and refer back to. I don't think we would need to return to the code to work on this version any more. |
@jessegreenberg, can you save me a version and post the link in this issue. (no rush) |
The above commits should address the problems you identified @terracoda. Here is a version with these changes that we can reference in the future: Once you have this version documented where you want it I think this can be closed @terracoda. |
Discussed today - good to have this version documented and we can close this issue and save to delete branches. |
Assigning to myself to delete branches and then close. |
Branch was deleted, closing. |
@jessegreenberg, I think I am done a first draft of the verbose and brief options for Interactive mode. Let me know if you have any questions.
Here's a link to the design doc:
https://docs.google.com/document/d/1XFTD2HTcGCEOP2yQ9_2SkEeXmCceZYUCmNeyiJfcCHs/edit#heading=h.6mboifb41tl8
The text was updated successfully, but these errors were encountered: