---- 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: 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

####

#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_

####

#2

Updated by Jim Nelson 11 months ago

  * **Target version** set to _0.14.0_

####

#3

Updated by Jim Nelson 11 months ago

  * **Category** set to _ux_

####

#4

Updated by Jim Nelson 10 months ago

  * **Category** changed from _ux_ to _import_

####

#5

Updated by Jim Nelson 9 months ago

  * **Target version** changed from _0.14.0_ to _0.15.0_

####

#6

Updated by Jim Nelson 8 months ago

  * **Target version** changed from _0.15.0_ to _0.16.0_

####

#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 for a partial solution that would at least let the user know the
import has not stalled.

####

#8

Updated by Clinton Rogers 7 months ago

  * **Status** changed from _Open_ to _Review_
  * **% Done** changed from _0_ to _70_

####

#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.

####

#10

Updated by Lucas Beeler 7 months ago

  * **Assignee** deleted (<strike>_Clinton Rogers_</strike>)

####

#11

Updated by Jim Nelson 6 months ago

  * **Target version** deleted (<strike>_0.15.0_</strike>)

####

#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