The 6 Hours speed plot only shows around 2 hours #4534

Closed
opened 2026-02-21 17:22:12 -05:00 by deekerman · 2 comments
Owner

Originally created by @itsdani on GitHub (Jul 18, 2016).

I haven't checked the exact time frame of the plot, but a point on the 6 hours plot is at ~1.25 grid when it goes out of the 30 Minutes plot, which means that the full 6 Hours plot shows only around 2 hours.

I have seen that there were recent issues with the 6 Hours plot CPU usage (#4120), maybe this issue is related to its fix (#4261) (I did not check the code though)

Versions:
Windows 8.1
qBittorrent 3.3.5
Qt 5.6.1
Libtorrent 1.0.9.0
Boost 1.61.0

Originally created by @itsdani on GitHub (Jul 18, 2016). I haven't checked the exact time frame of the plot, but a point on the 6 hours plot is at ~1.25 grid when it goes out of the 30 Minutes plot, which means that the full 6 Hours plot shows only around 2 hours. I have seen that there were recent issues with the 6 Hours plot CPU usage (#4120), maybe this issue is related to its fix (#4261) (I did not check the code though) **Versions:** Windows 8.1 qBittorrent 3.3.5 Qt 5.6.1 Libtorrent 1.0.9.0 Boost 1.61.0
Author
Owner

@ngosang commented on GitHub (Jul 18, 2016):

@sledgehammer999 ping.

@ngosang commented on GitHub (Jul 18, 2016): @sledgehammer999 ping.
Author
Owner

@itsdani commented on GitHub (Jul 21, 2016):

I've briefly looked into the code, and i think this causes the bug.
It should be m_counter6Hour = (m_counter6Hour + 1) % 18; instead of the % 6 (and not 36, since it has a double-sized buffer compared to the 30 min plot)

@itsdani commented on GitHub (Jul 21, 2016): I've briefly looked into the code, and i think [this causes the bug](https://github.com/onto/qBittorrent/blob/cdab0bb140b76083dbc77dea3caaeba445d08a37/src/gui/properties/speedplotview.cpp#L84). It should be `m_counter6Hour = (m_counter6Hour + 1) % 18;` instead of the `% 6` (and not 36, since [it has a double-sized buffer](https://github.com/onto/qBittorrent/blob/cdab0bb140b76083dbc77dea3caaeba445d08a37/src/gui/properties/speedplotview.h#L98) compared to the 30 min plot)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/qBittorrent#4534
No description provided.