Repository navigation
Conversation
Pure, GTK-free helpers for the upcoming transfer speed graph: the rounded y-axis ceiling, the moving average, bucketing of a long history into pixel columns, the overall average and the byte accounting of one operation (its own counter refined by the size of the file being written; a pause only moves the baseline, so a chunk finishing during a pause does not show up as one huge sample on resume). Unit tests cover all of it. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
GIO reports progress only every GiB or so with copy_file_range, which is too coarse to follow a transfer. Keep the byte count of the operation in the progress info, and let the copy code tell it which destination file is being written. The size of that file is read by a small worker thread pool, so a stalled network or FUSE mount can never block the main loop, and with lstat, so a symlink is not followed. For a plain copy or move that is the destination itself. When overwriting, GLib writes a .goutputstream-* temp file next to the target first; it is found among the process's own open descriptors and claimed per job, so two jobs in one directory never share one. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A collapsible graph under the "N of M" line of each file operation shows the transfer speed: a moving average over about a second, the whole history (up to an hour) stretched over the width, a rounded y-axis, dashed peak and average lines, and the current speed next to the buttons. A pause freezes the graph and the average instead of drawing zeros. The sampling interval comes from the new copy-speed-graph-interval setting (default 0.2 s, 0.05 to 5 s). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Between two reports from GIO, show the larger of the reported progress and the size of the file being written (never above 100%), so the bar moves smoothly while a big file is copied. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The details line now reads "Paused (X of Y)" instead of just "Paused". Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Author
|
Author
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a collapsible live transfer speed graph to the file operation progress window, makes the progress bar move smoothly while a big file is copied, and shows how much was copied while an operation is paused.
copy_file_range, GIO reports progress only every GiB or so, which is too coarse. The size of the file being written (the destination, or GLib's.goutputstream-*temp file when overwriting) is read by a small worker thread pool, withlstat, so a stalled network/FUSE mount can never block the main loop and symlinks are not followed. It is per job, so several running operations do not disturb each other.Paused (X of Y).copy-speed-graph-interval(default 0.2 s, range 0.05 to 5 s).The speed maths lives in a GTK-free module (
nemo-speed-graph-math) with unit tests.Notes
/proc/self/fd) and claims it per job. If it can't be found, the operation's own counter is used.Paused (%S of %S),peak: %s/s,avg: %s/s,now: ...,Transfer speed.Testing
Speed graph math test(13 cases) andProgress info live bytes test(live size, symlinks, directories, temp-file following, progress bar), clean under ASan/UBSan.test/need a display and a GSettings schema; they fail in a plain container with or without this change.🤖 Generated with Claude Code