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

Flip the Object and Context Responses around #245

Closed
terracoda opened this issue Nov 18, 2020 · 25 comments
Closed

Flip the Object and Context Responses around #245

terracoda opened this issue Nov 18, 2020 · 25 comments
Assignees

Comments

@terracoda
Copy link
Contributor

As discussed in the design meeting on Tuesday, we want to try and flip the object responses and context responses around for the hand interactions.

Reason, we like the order of the information when Lock Ratio check box is checked , and in that case, proximity to ratio is the value for the object responses.

I've sketched out an example CWT for the Both Hands interaction in this table in the design document.

The sliders will be similar, and in the case of the sliders there are also "distance progress indicators".

A few issues ago, I took the progress indicators out of the both hands interaction, but with the information flipped around, it might make sense to put them back. @zepumph, please do whatever is most straight forward, and then I can test and we can discuss.

@zepumph
Copy link
Member

zepumph commented Nov 24, 2020

@terracoda asked me not to do this until she could have a listen to some of the other changes I made in the description. @terracoda let me know when we are ready for this.

@terracoda
Copy link
Contributor Author

@zepumph, I made a table with a small test of the Left hand slider with the design flipped

I think it is safe to flip the design.

@terracoda
Copy link
Contributor Author

My test was applying the new logic, but we can discuss the logic before or after the flip - whatever makes sense to you.

@terracoda terracoda assigned zepumph and unassigned terracoda Dec 1, 2020
@terracoda
Copy link
Contributor Author

terracoda commented Dec 1, 2020

Oops, forgot to re-assign to you, @zepumph.

When you do this, I see 3 parts to the flip:

  1. Both Hands - looks like a direct flip since the ratio responses are currently on their own in context responses - rough examples in this table in the design doc
  2. Sliders - not a direct flip, since ratio responses are combined with distance (and progress) in context responses
  3. Adjustment of the current logic that replaces context responses with distance progress responses

Do you see it this way as well @zepumph?

@terracoda
Copy link
Contributor Author

@zepumph, I'll create separate issues numbers 2 and 3 above, in case that is helpful.
This issue can be for number 1.

@zepumph
Copy link
Member

zepumph commented Dec 4, 2020

This should be implemented, please review @terracoda.

@zepumph
Copy link
Member

zepumph commented Dec 5, 2020

When you review this, know that all flipping logic from side-issues (#261 and #262) has also been implemented.

@terracoda
Copy link
Contributor Author

with ratio locked, Moving hands down
Screen Shot 2020-12-07 at 6 07 33 PM

with ratio locked, Moving hands up
Screen Shot 2020-12-07 at 6 06 40 PM

@terracoda
Copy link
Contributor Author

@zepumph, I don't seem to get distance-progress if I have focus on a slider and ratio is locked. Here are three consecutive steps going up in the same distance region.

Screen Shot 2020-12-07 at 6 11 52 PM
Screen Shot 2020-12-07 at 6 12 09 PM
Screen Shot 2020-12-07 at 6 12 21 PM

@terracoda
Copy link
Contributor Author

Last comment is more appropriate for issue #262, maybe.

@terracoda
Copy link
Contributor Author

Oh, last 2 comments, #245 (comment) and #245 (comment)

maybe these comments should go in #262?
Anyways, this issue is connect to #261 and #262

@zepumph
Copy link
Member

zepumph commented Dec 9, 2020

From the design meeting today, @terracoda reiterated two bugs with the current implementation:

  • The distance-progress isn't working when ratio is locked
  • When using both-hands, the distance-progress is often (always!?!) backwards.

@zepumph zepumph assigned zepumph and unassigned terracoda Dec 9, 2020
zepumph added a commit that referenced this issue Dec 9, 2020
zepumph added a commit that referenced this issue Dec 9, 2020
…tance only when calculating description sentence, #245
@zepumph
Copy link
Member

zepumph commented Dec 9, 2020

The distance-progress isn't working when ratio is locked

This was fixed in 65abe56 by making sure that we only calculate the distance changed when we create the description. Before it was in a listener and could change multiple times before needing a value of "closer" or "farther", and it would be stale. This should be fixed now.

@zepumph
Copy link
Member

zepumph commented Dec 9, 2020

@terracoda, I believe that both bugs are fixed. I at least think it is ready for another review. Good luck!

@terracoda
Copy link
Contributor Author

Ok, cool.

@terracoda
Copy link
Contributor Author

  • closer/farther distance progress responses is no working for Both Hands with Lock Ratio checked.
  • with the hands unlocked, the Both Hands is getting distance progress responses

@terracoda
Copy link
Contributor Author

I still need to check with the tick marks, but don't anticipate issues.

@terracoda
Copy link
Contributor Author

  • the distance progress is working with the tick marks on.

With ratio locked, it seems when the left hand is visually "on a tick mark", it described as being "around" to tick mark.

Here's 2 screen shots. Not sure if this is directly related to this issue, but since I am testing this issue, I just need to post it here before I lose my train of thought:
Screen Shot 2020-12-09 at 4 15 54 PM
Screen Shot 2020-12-09 at 4 19 38 PM

@terracoda
Copy link
Contributor Author

Double checking with Lock Ratio unchecked.
Ok, this seems to happen only with Lock Ratio checked.

I can start out with Both Hands at 1:3 with each on the correct tick mark, but by the time the Left Hand gets to Tick Mark 2, the descriptions says "around tick mark 2" with right hand "on tick mark 6".

Also, if I start out at 1:3 and then move the hands down as far as they can go with ratio locked, and then move back up again, the left hand is described as "around tick mark 1" when it visually gets back on 1.

@terracoda
Copy link
Contributor Author

I commented over in #209 (comment) just in case the above comments should be in that issue.

@terracoda
Copy link
Contributor Author

terracoda commented Dec 9, 2020

@zepumph, in principle the correct object and context responses are firing for Both Hands.
The actual position of the left hand can become askew when ratio is locked.

Shall we work on that more in issue #209?
Assigning to you for closing.

@zepumph
Copy link
Member

zepumph commented Dec 9, 2020

I created #279 for that bug. It sounds like this issue about flipping responses is done. Closing

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

No branches or pull requests

2 participants