Skip to content

Conversation

@shyne99
Copy link
Contributor

@shyne99 shyne99 commented Jul 31, 2016

Voila. J'ai mis à jour le README, toutes les instructions sont dessus. Pour info ça ne marche que sur les mp4. Pour info j'ai fait mes test sur cette ID (5398676), et ça fonctionne sans soucis.

En cas de problème, verifie que node est toujours lancer, et la stdout de node affiche la grande partie des potentiels erreurs.

PS : C'est la première fois que je toucher à Django, donc si j'ai fait des choses qu'il ne faut pas, n'hésites pas à corriger ça.

Bon visionnage.

root and others added 11 commits July 25, 2016 16:14
…e un torrent depuis un magnet TODO// ajouter la partie client et tester la lecture pendant l'ecriture + ajout d'autres path afin de pouvoir voir la vitesse de DL, le status du DL, le pourcentage fait, l'ETA...
… le torrent id TODO : Passez à cette view les info de torrent, lancer le node par socket, et afficher la video via video.js
… for the moment) - have tu run sudo -H -u www-data bash -c 'node index.js' before connectiong to rufy and add "/stream to the end of the torrent_detail url
…wice before torrent already exist. We have now an up to date list of torrent in the API
@shyne99
Copy link
Contributor Author

shyne99 commented Jul 31, 2016

PS : Cela n'a pas l'air de marcher sur chrome, peut être une extension qui bloque le flux.. Fonctionne sur firefox

@MisterCyp
Copy link
Owner

Salut Shyne,

J'ai testé mais, je ne sais pas si c'est à cause de ma VM, mais la lecture est super longue et donc inutilisable :/. Mais en tout cas la preuve de concept est la :) merci beaucoup !!
Et donc avec ce système, seul les MP4 peuvent être lus ?

Je te mets les erreurs node que j'ai eu si ça peut t'aider pour le debugage :

node /var/www/RuFy/nodejs/index.js
Listening on http://127.0.0.1:3000
{}
Removing torrent error : TypeError: path must be a string or Buffer
Torrent added
(node:1686) Warning: Possible EventEmitter memory leak detected. 11 ready listeners added. Use emitter.setMaxListeners() to increase limit
{}
Removing torrent error : TypeError: path must be a string or Buffer
Torrent added
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: Cannot add duplicate torrent b6df181160fef6778f37a1620a22d7001c1ac65a
    at Torrent.onInfoHash (/var/www/RuFy/nodejs/node_modules/webtorrent/index.js:262:26)
    at Torrent.g (events.js:286:16)
    at emitOne (events.js:96:13)
    at Torrent.emit (events.js:188:7)
    at Torrent._onParsedTorrent (/var/www/RuFy/nodejs/node_modules/webtorrent/lib/torrent.js:263:8)
    at /var/www/RuFy/nodejs/node_modules/webtorrent/lib/torrent.js:230:12
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)
rufy@debian:/$ node /var/www/RuFy/nodejs/index.js
Listening on http://127.0.0.1:3000
Torrent added
(node:1701) Warning: Possible EventEmitter memory leak detected. 11 ready listeners added. Use emitter.setMaxListeners() to increase limit
WebTorrent: `torrent.swarm` is deprecated. Use `torrent` directly instead.
{ id: '/tmp/Vice Versa 2015 Truefrench 720p x264 AAC PIXEL.mp4.torrent' }
Removed torrent/tmp/Vice Versa 2015 Truefrench 720p x264 AAC PIXEL.mp4.torrent
Torrent added
WebTorrent: `torrent.swarm` is deprecated. Use `torrent` directly instead.

Je viens de penser aussi que ce n'était pas forcement utile de push ta branche sur mon repo, on peut la laisser sur le tien en attendant une version plus stable ;)

Merci encore pour ton travail ;)

@shyne99
Copy link
Contributor Author

shyne99 commented Aug 3, 2016

Ca marche. J'ai fixé plusieurs bug déja (nottament le Cannot add duplicate..), et j'ai ajouter un fichier de conf pour init.s afin de le lancer en servcie. Mais il est vrai que ce n'est qu'une preuve de concept. Je continue le travail la dessus. Je pense partir sur un encodage à la volée des fichiers par ffmpeg. Dès que j'ai quelque chose de concluant et que j'aimerai que quelqu'un d'extérieure test tout ça je te ferai signe.

Merci.

@shyne99
Copy link
Contributor Author

shyne99 commented Aug 8, 2016

Salut,

Je viens de finir ma V0, j'ai corrigé beaucoup de bug et j'ai fini la première version qui transcode les torrent à la volée.
Pourrais tu essayer cette version et me dire si tu rencontre des problèmes ? la branche a changé, elle s'appelle "transcode".
Je l'ai essayé pendant un moment, et je n'ai pas eu de plantage. Par contre le transcodage est réglé par défaut en haute qualité, car il tourne sur mon serveur.
Si tu as l'image et le son, et que ça saccade, c'est juste l'encodage qui demande trop de ressources.

Si tu peu me faire un retour la dessus quand tu auras essayer, ce serai sympa.

Merci.

@MisterCyp
Copy link
Owner

Salut,

Alors j'ai essayé ta nouvelle version. C'est vraiment pas mal même si j'ai eu des problèmes lors de l'installation. Lorsque tu définies l'adresse ip du serveur NodeJS, il prend l'adresse ip local et du coup, ca ne marche pas sur un serveur distant. J'ai du mettre l'adresse de mon serveur à la place (https://github.com/shyne99/RuFy/blob/transcode/t411/templates/t411/stream_torrent.html#L11).

D'ailleurs quand tu fais tes tests, c'est sur un serveur distant ou local ? Car sur un serveur distant, la lecture est censée être plus longue ...

Sinon lorsqu'il n'y a pas de transcodage, la lecture est parfois fluide, parfois saccadée. Pour une deuxième lecture du même torrent, la lecture est fluide. D'ailleurs à chaque fois que je voulais lire un autre torrent, j'étais obligé de relancer le serveur nodeJS ...

Par contre, lors du transcodage, la lecture est saccadée mais je n'ai pas fait le test en baissant la qualité. J'ai utilisé le VPS VC1S de chez Scaleway (https://www.scaleway.com/) pour faire le test (J'arrive à faire du transcodage avec EMBY sans problème avec ...).
D'ailleurs, si tu as le temps/la possibilité de développer une image docker de ta branche, ce serait cool. C'est beaucoup plus simple pour la tester ensuite.

Merci encore pour ton travail.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants