Merge lp:~fred-bertolus/luciole/fix-pitivi-export into lp:luciole

[Fred Bertolus]

Bonjour,

En cherchant un logiciel pour faire du timelapse, je suis tombé sur luciole que semblait convenir. Malheuresement, l'export pitivi ne générait pas les bonnes durées pour les images (par exemple chaque images duraient 0.5 seconde au lieu de 0.08 seconde pour un framerate de 12,5 images par secondes.

J'ai donc téléchargé la version de dev, qui avait le même comportement, mais la boite de dialogue ne s'affichait plus.

J'ai donc fait 2 corrections :

- Réparation de l'affichage de la boite de dialogue "export tools"
- Correction de la durée des images dans l'export Pitivi

Merci pour le logiciel

[NicoInattendu (nico-inattendu)]

To be linked with Bug #782641. For the export tool problem
To be linked with Bug #782643 : for export to Pitivi problem

review: Approve

=== modified file 'luciole/gui/actions.py'
--- luciole/gui/actions.py	2011-03-25 09:19:59 +0000
+++ luciole/gui/actions.py	2011-05-14 07:42:32 +0000
@@ -39,6 +39,7 @@
 import luciole.gui.constants as GUI_CONST
 import luciole.constants as LCONST
 from luciole.gui.windows.export_video_window import ExportVideoWindow
+from luciole.gui.windows.export_tool_window import ExportToolWindow
 from luciole.info import REVNO, VERSION
 _tv_change_signal = 'treeview-change'
 _tv_selected_signal = 'treeview-selected'
@@ -472,8 +473,8 @@
         """ report bug action """ 
         self.debug('Report bug action')
         try:
-			webbrowser.open("https://bugs.launchpad.net/luciole/+filebug")
-		except:
+            webbrowser.open("https://bugs.launchpad.net/luciole/+filebug")
+        except:
             _msg = _("Unable to open the Launchpad web page.")
             self.warning(_msg)
             self.gui.windows.error_message(_msg)
@@ -482,8 +483,8 @@
         """ ask a question action """ 
         self.debug('Ask a question action')
         try:
-			webbrowser.open("https://answers.launchpad.net/luciole/+addquestion")
-		except:
+            webbrowser.open("https://answers.launchpad.net/luciole/+addquestion")
+        except:
             _msg = _("Unable to open the Launchpad web page.")
             self.warning(_msg)
             self.gui.windows.error_message(_msg)
@@ -492,8 +493,8 @@
         """ Help trranslate action """ 
         self.debug('Help translate action')
         try:
-			webbrowser.open("https://translations.launchpad.net/luciole")
-		except:
+            webbrowser.open("https://translations.launchpad.net/luciole")
+        except:
             _msg = _("Unable to open the Launchpad web page.")
             self.warning(_msg)
             self.gui.windows.error_message(_msg)
@@ -537,8 +538,10 @@
         _export.run(self.app.project_ctrl.project.props)
 
     def _export_tool_cb(self, action) :
-        # TODO implement
-        pass
+        """ Export tools callback"""
+        _export_tools = ExportToolWindow(project=self.app.project_ctrl.project.props)
+        _export_tools.run()
+        
      
     def _project_properties_cb(self, action) :
         """ Display of project properties"""
 
=== modified file 'luciole/project/export/export_pitivi.py'
--- luciole/project/export/export_pitivi.py	2010-04-22 07:05:32 +0000
+++ luciole/project/export/export_pitivi.py	2011-05-14 07:42:32 +0000
@@ -56,7 +56,8 @@
     __STREAM_NAME = "src0"
     __STREAM_TYPE = "pitivi.stream.VideoStream"
 
-    __BASE_DURATION_NS = 1000000000
+    __BASE_DURATION_SECOND = 1000000000
+    __BASE_FRAME_RATE = 25
 
     __TIMELINE_TYPE = "pitivi.timeline.track.SourceTrackObject"
     __TIMELINE_IN_POINT = "(gint64)0"
@@ -166,7 +167,7 @@
         """ compute duration in pitivi format. 
             Input is the fpi : number of frame per image
         """
-        duration = (1.0/fpi)*self.__BASE_DURATION_NS
+        duration = fpi*self.__BASE_DURATION_SECOND / self.__BASE_FRAME_RATE
         # use of str and int : int for a rounded value and str for string in element tree
         return str(int(duration))