Sink controls redesign
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- audio-video-rtc
Línea de trabajo
Read src/sink.rs and the control implementation in src/source/amplify.rs, then trace the callback setup referenced in src/source.rs. The issue proposes composable per-input controls and faster parameter updates, but leaves the design open; compare atomic values with per-converter callbacks and determine how the chosen approach meets the goals described.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
A follow up on https://github.com/RustAudio/rodio/issues/619#issuecomment-2372360582
Current implementation of Sink unconditionally includes several audio controls like pause, volume, speed (see src/sink.rs). There are several things that can be improved over current implementation.
Users may not need all for these at once and prefer to have only necessary conversions. It is possible to make the controls composable so users can add only necessary parts and adjust audio parameters for each input separately.
Current implementation of controls (like volume control, see here ) only accepts fixed values. Sink allows to control these parameters dynamically, which is implemented with a callback that is invoked periodically. The callback applies values requested by user to the actual control. Currently this happens at about 5ms intervals.
It is possible to apply the parameters directly by using atomic values. This should reduce reaction time and simplify the 'Sink' implementation (see callback setup in src/source.rs). In case atomics turn out to be expensive on some architectures, it is still possible to simplify Sink implementation by allowing each audio converter to control its own parameters. In that case each converter will have to have its own periodic_callback.
Should be included into #619.
- Lenguaje dominante
- Rust
- Estrellas
- 2.5k
- Forks
- 338
- Merge medio
- 1 d 3 h
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de RustAudio/rodio
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 73/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 62/100
-
WavDecoder::size_hint underflows (u32 wrap, ~4.29e9) after EOF, causes multi-GB Vec::collect() allocationPosiblemente ocupada @marin-m la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Todos los issues de RustAudio/rodio
Issues similares
-
desktop documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
vfarcic/dot-agent-deck#2008 ·
Los mantenedores suelen responder en 1 día
-
needs triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
pbakaus/impeccable#1210 ·
Los mantenedores suelen responder en 1 día
-
area: ci priority: P4 ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
shep-pm/shep#1249 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
L: javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
dependabot/dependabot-core#16744 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
nautechsystems/nautilus_trader#5486 · 1 comentario ·
Los mantenedores suelen responder en 1 día