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

Metronomo #16

Open
rvega opened this issue Apr 2, 2014 · 4 comments
Open

Metronomo #16

rvega opened this issue Apr 2, 2014 · 4 comments

Comments

@rvega
Copy link
Member

rvega commented Apr 2, 2014

Johhnny, hay varios asuntos con el metrónomo:

  1. Recordá quitar los elementos de GUI o prints o lo que sea, despues de hacer debug en tus patches de DSP.
  2. El blogue dsp debe ser un abstraction y no un subpatch. Para poderlo llevar al mikro-syhtn.dsp.
  3. En la GUI solo debe haber un botón (bang) y un slider. Sobra lo de ON/OFF.
  4. Cuando le doy click 4 veces al tap, el metronomo debe empezar a funcionar. No esperar al slider.
  5. El metronomo debe arrancar encendido, en un bpm default (120bpm).
  6. Debes conectar el metronomo al resto del sinte y jugar con el un ratico antes de declararlo "listo". Esto para detectar y arreglar bugs que sean obvios cuando tenés la implementación de tu módulo fresca en la mente. Hacer esto en el branch bpm y no hacer merge a master hasta que otro de nosotros lo haya mirado. (lo mismo para todos nosotros).

Si podés arreglar esto hoy, magnífico.

@rvega
Copy link
Member Author

rvega commented Apr 2, 2014

  1. Los rangos del slider y su denormalize están distintos.

@Kaziuz
Copy link

Kaziuz commented Apr 2, 2014

Rafa me podrías decir por favor cuando estoy en la rama, como puedo descargar el patch, cuando cree la rama en mi compu y subi el patch, este mismo desapareció de mi local.
Gracias

@rvega
Copy link
Member Author

rvega commented Apr 2, 2014

chat

@rvega rvega added this to the Sintetizador en PD milestone Apr 7, 2014
@rvega
Copy link
Member Author

rvega commented Apr 7, 2014

Falta hacer merge a master y testear esto bien.

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

No branches or pull requests

2 participants