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

Tests sometimes register inputs in the next frequency band #4

Closed
MoralCode opened this issue Jun 23, 2023 · 3 comments
Closed

Tests sometimes register inputs in the next frequency band #4

MoralCode opened this issue Jun 23, 2023 · 3 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@MoralCode
Copy link

MoralCode commented Jun 23, 2023

test_BOTH_5-22-123

as a hearing user wearing bluetooth headphones, I gave this a try and noticed that, because i was able to hear the tones until they were rather quiet (i.e. low y values), i would sometimes click the spacebar too late and it would register the input for the next tone. (that is at least until i got bored a few frequencies in and started spamming the spacebar as you can see in the data haha)

I feel like it might help produce better data if the app waits for user input before moving onto the next tone

Seems like a cool app though! might also make for a fun tool for hearing users to compare different pairs of headphones and things like that with a little more empirical rigor than just subjective listening

@Jah-On
Copy link
Owner

Jah-On commented Jun 23, 2023

Solution proposal: Omit the auto reset system and instead notify the user with a text node that auto hides/shows telling them the pitch has reached 0 and to either press space if they could hear the pitch or 0 if they could not. Does this seem like a reasonable proposal?

@Jah-On Jah-On added bug Something isn't working enhancement New feature or request labels Jun 23, 2023
@MoralCode
Copy link
Author

Yeah, i was sorta thinking of something like that as well!

@Jah-On
Copy link
Owner

Jah-On commented Jun 23, 2023

Fixed in 53a6657, thanks!

@Jah-On Jah-On closed this as completed Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants