miraclx/freyr-js

Transcoding to 320kbps is useless

Open

#440 aperta il 24 feb 2023

Vedi su GitHub
 (4 commenti) (7 reazioni) (0 assegnatari)JavaScript (154 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (2279 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Because the audio is sourced from YouTube the highest bitrate audio available is 256kbps. Transcoding the already compressed 256kbps audio to 320 kbps AAC will only result in loss of information as AAC is a lossy codec and transcoding is generally an at least somewhat lossy process. I suggest instead downloading the maximum of 256kbps and leaving it be, or if the user specifies a lower bitrate downloading YouTube's copy of the audio at that bitrate (which much of the time does exist). If no such version can be found, download the 256kbps copy and transcode to the lower bitrate.

Guida contributor

Direzione di ricerca
Esamina la logica di download nel codice sorgente (probabilmente in src/ o lib/), concentrandoti su come viene selezionata e transcodificata la bitrate audio. Cerca il codice che gestisce il recupero dell'audio da YouTube e eventuali flag come bitrate. Quindi modifica la logica per evitare la transcodifica a 320kbps quando l'input è 256kbps, mantenendo l'originale o transcodificando verso il basso se viene richiesta una bitrate inferiore. Verifica se esistono test per la transcodifica e aggiungi un caso di test che verifichi l'assenza di transcodifica non necessaria.
Tech stack
javascriptnodejs
Dominio
clibackend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75