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

Add guitar tuner #976

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Conversation

twollgam
Copy link
Contributor

@twollgam twollgam commented Mar 23, 2024

What does this PR do?

This pr adds a guitar tuner screen to performous. It is an alpha feature that must be enabled in the configuration.

It has two objectives:

  1. Tune a six string guitar ;-) As I want to add real guitars as instruments in the future this is a very basic start.
  2. Test the current note/tone detection.

Closes Issue(s)

None.

Motivation

I like to add real guitars as instruments and this is a very basic start.

More

Nothing yet.

Additional Notes

It seems that my impression that deep notes are not exactly detected will be proved. Currently all strings are detected right but the deep e string is not.

@twollgam twollgam self-assigned this Mar 23, 2024
@Baklap4
Copy link
Member

Baklap4 commented Mar 29, 2024

I think this is still a draft and doesn't need review yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants