miraclx/freyr-js

High memory usage

Open

#646 ouverte le 23 janv. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (2 279 stars) (154 forks)user submission
bughelp wanted

Description

Thank you for developing this app. I appreciate its user-friendliness.

However, I noticed that when downloading multiple large playlists and albums using queue, the app's memory usage spikes significantly. I wanted to point this out so that you can consider optimizing the memory management for these scenarios.

2024-01-23_16-29

Command used: npx freyr get https://open.spotify.com/artist/4CyxMgnF71FyOWADVWxJRE?si=ryq2taQMQtmXns5eyoJtoA https://open.spotify.com/artist/1v7B6ZWa7QRQS3knn3Jvf4 https://open.spotify.com/playlist/37i9dQZF1DWXgtgITTKvYl https://open.spotify.com/playlist/37i9dQZF1DX5wgKYQVRARv

Guide contributeur

Stack technique
javascriptnodejs
Domaine
cli
Type d'issue
performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Node.js memory profilingCLI tool debugging
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25
Direction de recherche
Investigate memory usage spikes during concurrent downloads. Use Node.js heap snapshot tools (e.g., inspect) to identify objects retaining memory. Look at the download queue implementation in the source code (likely in src/ directory). Check for any unstreamed responses or caches growing unboundedly.