Include progress bar when importing a large video
Submitted by an unknown user
Link to original bug (#718494)
Description
---- Reported by shotwell-maint@gnome.bugs 2012-01-06 16:05:00 -0800 ----
Original Redmine bug id: 4571
Original URL: http://redmine.yorba.org/issues/4571
Searchable id: yorba-bug-4571
Original author: Laura Khalil
Original description:
Reported downstream on Launchpad: https://bugs.launchpad.net/ubuntu/+source/shotwell/+bug/912964
These days, more and more digicams ship with video capabilities. These videos can become huge, multiple gigabytes for one video is not an exception.
However, Shotwell seems to be loading the entire video into memory before writing it to disk, which completely 'freezes' the import process (there's no indication of the fact that a video is being copied and nothing appears to be happening), slows down the computer since most of the internal memory is being consumed and it potentially crashes machines which are low on memory.
What should happen is that:
a) the user gets informed by Shotwell that a video is being imported (text feedback with i.e. file name/size + ETA, progress bar, thumbnail of the video)
b) instead of loading the entire file into memory, Shotwell should load 'reasonable' chunks at a time and write those to disk sequentially.
ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: shotwell 0.11.6-0ubuntu0.1 [modified: usr/share/glib-2.0/schemas/gschemas.compiled]
ProcVersionSignature: Ubuntu 3.0.0-14.23-generic 3.0.9
Uname: Linux 3.0.0-14-generic x86_64
ApportVersion: 1.23-0ubuntu4
Architecture: amd64
Date: Mon Jan 2 21:42:16 2012
InstallationMedia: Ubuntu 11.10 "Oneiric Ocelot" - Release amd64 (20111012)
SourcePackage: shotwell
UpgradeStatus: No upgrade log present (probably fresh install)
A progress bar seems like a reasonable request in this case.
Related issues:
- related to shotwell - Feature #2035 (closed): advance progress bar smoothly as each RAW photo is imported (Open)
- related to shotwell - 4138: Frequently, sluggishness occurs when merging tags. (Open)
---- Additional Comments From shotwell-maint@gnome.bugs 2013-05-30 11:12:00 -0700 ----
History
Comment 1
Updated by Laura Khalil almost 2 years ago
- Subject changed from Include progress bar when importing a large file to Include progress bar when importing a large video
Comment 2
Updated by Jim Nelson 11 months ago
- Target version set to 0.14.0
Comment 3
Updated by Jim Nelson 11 months ago
- Category set to ux
Comment 4
Updated by Jim Nelson 10 months ago
- Category changed from ux to import
Comment 5
Updated by Jim Nelson 9 months ago
- Target version changed from 0.14.0 to 0.15.0
Comment 6
Updated by Jim Nelson 8 months ago
- Target version changed from 0.15.0 to 0.16.0
Comment 7
Updated by Jim Nelson 8 months ago
- Category changed from import to ux
- Priority changed from High to Normal
- Target version changed from 0.16.0 to 0.15.0
See #2035 (closed) for a partial solution that would at least let the user know the import has not stalled.
Comment 8
Updated by Clinton Rogers 7 months ago
- Status changed from Open to Review
- % Done changed from 0 to 70
Comment 9
Updated by Lucas Beeler 7 months ago
- Status changed from Review to Open
- Assignee set to Clinton Rogers
- Priority changed from Normal to High
Review: reject. I've just spent quite a bit of time testing this and I have to say that the user experience is not satisfactory. What I've observed is this. Let's say I start importing a video that takes 45sec to import. In this case, I see the progress bar pulse for about 15 seconds and then pulsing ceases and we're right back to the old behavior. This is a thornier problem than we originally anticipated. In this patch we're spinning the event loop from inside a Timeout invoked via the event loop. I'll have to think about it more deeply but my gut says that this has something do with the problem I've described whereby the "pulsing" progress bar stops pulsing.
Comment 10
Updated by Lucas Beeler 7 months ago
-
Assignee deleted (
<strike>
_Clinton Rogers_</strike>
)
Comment 11
Updated by Jim Nelson 6 months ago
-
Target version deleted (
<strike>
_0.15.0_</strike>
)
Comment 12
Updated by Fabio Bonfante 6 months ago
A really little small improvement for a 0.14.x version could be just to (pseudocode)
if (fileToImport.Size > 5MB) {
progressbarLabel.Text = "Importing " + fileToImport.Name + " will take a while..."
}
--- Bug imported by chaz@yorba.org 2013-11-25 21:57 UTC ---
This bug was previously known as bug 4571 at http://redmine.yorba.org/show_bug.cgi?id=4571
Unknown version " in product shotwell. Setting version to "!unspecified". Unknown milestone "unknown in product shotwell. Setting to default milestone for this product, "---". Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one. Resolution set on an open status. Dropping resolution