Ratio incorrect pour les torrents seedés directement

Bug #522976 reported by phorane
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
qBittorrent
Fix Released
Medium
Christophe Dumez

Bug Description

Je sais que ça a déjà été signalé mais sur une version plus ancienne et qui a été signalée comme «Fix Released» alors que le problème reste toujours le même.

Lorsqu'on commence à seeder directement un fichier (donc sans le télécharger), le ratio est d'abord mis à 0 tant que rien n'a été envoyé et passe ensuite à l'infini.

Et non ce n'est pas un comportement normal (contrairement à ce qui a été dis dans un autre rapport de bug) à mon avis, le ratio doit être calculé par rapport à total_done et non pas total_download, en tout cas c'est comme ça que font tous les autres clients que j'ai déjà utilisé, y compris le client officiel (et aussi les trackers).

De toute manière un ratio de partage infini n'a aucun sens, et certainement pas celui de dire que lorsqu'on n'a rien téléchargé alors on a partagé à l'infini !

Donc pour moi C'EST un bug, non seulement dans qbittorrent, mais aussi dans btfaq citée ailleurs (http://btfaq.com/serve/cache/23.html) (car oui il y a aussi des bugs dans les specs -- qui expliquent une bonne partie des bugs dans les programmes)

Et puis il faut être pragmatique: en étant le seul client à se référer a une telle définition, tu n'as pas fini d'avoir des rapports de bugs là dessus. Et deuxième raison pragmatique: à quoi ça sert d'avoir un truc qui reste bloqué sur l'infini? Personnellement (et je pense pour beaucoup d'autres) le ratio me sert à savoir qu'est-ce que j'ai envoyé (1, 2, 3, 10 fois le fichier?) et quand c'est moi qui débute le seed c'est une information que je veux vraiment avoir. Hors si on active le DHT par exemple, ce n'est pas le (ou les) tracker qui pourra donner une information correcte là-dessus, seulement mon client bt.

C'est très simple à corriger: (par rapport aux sources de la version 2.2.0beta4) remplacer dans Bittorrent::getRealRatio les 3 occurrences de all_time_upload par total_done et donc ajouter total_done à qtorrenthandle.h et qtorrenthandle.cpp pusiqu'il n'y ait pas déjà.

je met un patch sur la version 2.2.0beta4 en pièce jointe.

Ce serait quand même dommage que ce soit ceux qui sont capables de corriger eux-mêmes ce bug qui en profite alors que ce sont ceux qui en ont le plus besoin qui ne pourront pas le corriger.

On va quand même pas faire un fork pour ça !

Si ça te dérange à ce point de contrarier la btfaq alors ajoute au moins une colonne supplémentaire qui donne le «ratio de seed» (?) calculé avec total_done qui est le seul réellement intéressant.

Tags: ratio
Revision history for this message
phorane (phorane) wrote :
Changed in qbittorrent:
status: New → Confirmed
assignee: nobody → Christophe Dumez (hydr0g3n)
Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

Bonjour,

Je n'ai plus souvenir d'un ancien rapport de bug a ce sujet (ou j'aurais refusé de corriger ce bug). Ce rapport doit dater et j'avais peut etre mal compris le probleme.

En tout cas, je suis d'accord qu'il s'agit d'un bug et je vais appliquer ton patch en rentrant du boulot ce soir.

P.S: Au lieu de parler de fork, rejoins plutot l'equipe de dev qui comporte pour le moment un seul et unique developpeur (moi). Quand on est tout seul pour faire le developpement, s'occuper des rapports de bugs, de la documentation, des releases, des reponses aux questions : c'est sur que c'est plus facile de faire des erreurs.

Changed in qbittorrent:
importance: Undecided → Medium
milestone: none → 2.2.0
Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

Ok. C'est corrigé dans SVN (trunk et branche stable). Ce sera donc dans la prochaine release.

Changed in qbittorrent:
status: Confirmed → Fix Committed
summary: - un ratio utilisable ce serait bien
+ Ratio incorrect pour les torrents seedés directement
Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

Phorane > Next time, please file your reports in English if you can. French is no problem for me but there are other people who receive the bug reports (like packagers) and they do not necessarily understand French.

Leigh > This patch fixes the calculation of the ratio for the torrents that are directly seeded (without being downloaded using qBitttorrent first).

Revision history for this message
phorane (phorane) wrote :

Il s'agissait effectivement d'un vieux rapport de bug (#364730) mais qui portait sur la même chose mais sous l'angle d'une de ses conséquences (le fait que les torrents mis directement en seed se retrouvait directement effacé avec l'option «Supprimer les torrents terminés lorsque leur ratio atteint».

Bref, l'histoire du fork était juste une plaisanterie mais dans ce rapport de bug tu avais l'air de dire que ce comportement était on ne peut plus normal et que tu ne voulais pas le modifier.

Sinon désolé pour le français mais je ne suis pas forcément capable d'écrire tout ça en anglais facilement alors à choisir entre ne rien signaler et le signaler en français... bah j'ai choisi

Sinon pour rejoindre l'équipe de dev je suis pas certain que je serai un cadeau pour toi parce que à la base je ne suis pas vraiment développeur mais spécialisé en méthodes formelles pour le développement de logiciels/systèmes informatiques... bref je suis pas du genre à me contenter d'un «ça marche parce qu'on voit bien que ça marche et s'il ne le faut on modiefera plus tard», et je peux donc être assez casse-c... Mais si ça t'intéresse d'avoir un coup de main je pense que tu as mon email, j'ai beaucoup de temps en ce moment et j'aime bien qbittorrent et j'y apporterais volontiers des améliorations.

Changed in qbittorrent:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.