After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 581997 - Patch to support taking a series of photos
Patch to support taking a series of photos
Status: RESOLVED FIXED
Product: cheese
Classification: Applications
Component: general
2.24.x
Other Linux
: Normal normal
: 2.26
Assigned To: Cheese Maintainer(s)
Cheese Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-05-09 16:17 UTC by Aidan Delaney
Modified: 2009-07-29 22:32 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
Allow cheese to take a series of photos. (6.66 KB, patch)
2009-05-09 16:17 UTC, Aidan Delaney
reviewed Details | Review
Allow cheese to take a series of photos. (11.53 KB, patch)
2009-07-14 21:01 UTC, Aidan Delaney
none Details | Review
Burst mode plumbing for cheese (5.67 KB, patch)
2009-07-27 16:00 UTC, Aidan Delaney
none Details | Review
Now with added gconf schema support (6.66 KB, patch)
2009-07-27 16:35 UTC, Aidan Delaney
committed Details | Review

Description Aidan Delaney 2009-05-09 16:17:06 UTC
This patch does the plumbing to allow cheese to take a series of photos with a delay between them.  It does not provide UI configuration for the GConf keys it installs.

It's based on the origin/gnome-2-24 branch in git.
Comment 1 Aidan Delaney 2009-05-09 16:17:38 UTC
Created attachment 134314 [details] [review]
Allow cheese to take a series of photos.
Comment 2 Aidan Delaney 2009-05-09 19:28:10 UTC
Would it make sense to re-implement this feature as a WEBCAM_MODE_TIMELAPSE?
Comment 3 daniel g. siegel 2009-05-10 15:56:18 UTC
great work!

hmm.. do you have any idea, how we could integrate this into the ui?
Comment 4 Aidan Delaney 2009-05-10 20:04:10 UTC
I was thinking that you'd take a Cheese->"timelapse photographs".  This would be similar to the "Photo" or "Video" buttons on default cheese window and implemented as WEBCAM_MODE_TIMELAPSE.  However, if it was undesirable to have timelapse functionality supported as a first-class citizen (as the function may be used rarely) I'd bury the options in Edit->Preferences and have the options apply to the default "Take a Photo" i.e. WEBCAM_MODE_PHOTO with repeat==true.  As an intermediate option between the two, I'd implement this as Cheese->timelapse with no button on the default cheese window i.e. discoverable - but not cluttering the defaults.

I'm happy to complete implementation of the feature as directed by Cheese-central.  Please advise.
Comment 5 daniel g. siegel 2009-05-10 20:56:31 UTC
hehe, we do not have a cheese central ;)

anyway, i think both ways are possible. would you mind to join our mailing list and propose it there? we will get a few more voices and then we can decide. sounds like a plan?
Comment 6 daniel g. siegel 2009-06-22 11:01:54 UTC
any news on this?
Comment 7 Aidan Delaney 2009-06-23 13:40:32 UTC
Sort of.  I've got the following

http://gitorious.org/cheese-lapse/cheese-lapse/commit/b6efbb8fd22806792ad5e394c5e7b752e5147a94

I've added a user_framerate as a "property" of the webcam.  If the user does not set the framerate then the user_framerate becomes the highest_framerate, otherwise the camera captures at its native framerate but produces a video at user_framerate.

I've not exposed a "Take [1 picture |V] every [1 second |V]" UI thing yet.  So the user cannot currently set the user_framerate.  I intend to add this to the right of the "Start recording" button.

I need to do some more testing.  And I need to make sure my UI modifications work on Hildon etc...
Comment 8 daniel g. siegel 2009-06-25 09:36:51 UTC
looks quite good! please go ahead ;)
Comment 9 Tobias Mueller 2009-07-11 21:34:15 UTC
Hey Aidan, could you please cook up a patch soon, because 2.28 is approaching fast and we'd love to get this in :) Thanks in advance!
Comment 10 Aidan Delaney 2009-07-12 09:26:54 UTC
I merged your main branch onto my cheese-lapse branch here
http://gitorious.org/cheese-lapse/cheese-lapse/commits/cheese-lapse
as *I think* I was running into the x_overlay bug.

Testing required.  I need to upgrade my Fedora install in order to do this.
Comment 11 Aidan Delaney 2009-07-12 18:47:47 UTC
After a little more hacking I conclude that I have no idea what I'm doing.  Or at least, a have a little clue.  The UI bits work.  And I'm twiggiling the correct framerate settings.  But both don't work together.

More beer required, will get back to it tomorrow.

Is there a cheese IRC channel?
Comment 12 daniel g. siegel 2009-07-13 00:53:27 UTC
no, we do not have one. but you can add me on jabber (same mail address).

actually, we probably should slightly alter your idea to the following: what about taking a series of _photos_ instead of a video? as in the ui, maybe a small button on the right of the "take a photo" button would be apprpriate, where just a few options are listed, like: 3,5, 10. the time between those photos can be set then inside gconf. what do you think?
Comment 13 Aidan Delaney 2009-07-14 21:01:25 UTC
Created attachment 138412 [details] [review]
Allow cheese to take a series of photos.

This solves the bug in the manner that you suggest.  Taking a series of photos is _much_ easier than doing a timelapse video.
Comment 14 Aidan Delaney 2009-07-14 21:02:19 UTC
I should say that this is based on today's git.  And the UI may need a bit of spit and polish.
Comment 15 daniel g. siegel 2009-07-14 21:57:48 UTC
the patch works great by itself, but as you said we need to polish the ui. my first idea would be to add a small dropdown bar on the right of the "take a photo" button (but still inside the take a photo alignment box") ?

do you have other ideas?

thanks a lot for your great effort!
Comment 16 Aidan Delaney 2009-07-15 10:47:59 UTC
The other way I can see to do it is via the Edit->Preferences or we could have a PhotoBlast mode. Which does as you suggest.

I personally don't like clogging up the default UI with a fugly spinbutton.  So the PhotoBlast mode with the setting in Edit->Preferences makes sense to me.
Comment 17 Aidan Delaney 2009-07-27 16:00:12 UTC
Created attachment 139293 [details] [review]
Burst mode plumbing for cheese

This allows countdowns after a delay and supports full-screen.
Comment 18 Aidan Delaney 2009-07-27 16:35:29 UTC
Created attachment 139296 [details] [review]
Now with added gconf schema support

I forgot to modify cheese.schema.in
Comment 19 daniel g. siegel 2009-07-27 22:21:17 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 20 Tobias Mueller 2009-07-29 22:22:30 UTC
Hey daniel, can you tell us the revision which made the patch?
That helps for ISVs backporting the patch, finding the commited patch in case anybody wants to follow up here, etc.
TIA!