unable to separate video and audio by using "ungroup clips" button

Bug #1100007 reported by Experimenter
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pitivi (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

unable to separate video and audio by using "ungroup clips". On clicking "ungroup clips' button nothing happens
ver 0.15.2-0ubuntu0.1 in 12.04 LTS

Below is the traceback thrown on clicking ungroup clips
Traceback (most recent call last):
  File "/usr/lib/pitivi/python/pitivi/ui/timeline.py", line 764, in ungroupSelected
    self.timeline.ungroupSelection()
  File "/usr/lib/pitivi/python/pitivi/timeline/timeline.py", line 1952, in ungroupSelection
    self.selection.setSelection(timeline_object.track_objects, UNSELECT)
  File "/usr/lib/pitivi/python/pitivi/timeline/timeline.py", line 527, in setSelection
    self.emit("selection-changed")
  File "/usr/lib/pitivi/python/pitivi/signalinterface.py", line 151, in emit
    *args, **kwargs)
  File "/usr/lib/pitivi/python/pitivi/signalinterface.py", line 126, in emit
    res = cb(*ar, **kw)
  File "/usr/lib/pitivi/python/pitivi/timeline/timeline.py", line 1548, in _selectionChanged
    self.emit("selection-changed")
  File "/usr/lib/pitivi/python/pitivi/signalinterface.py", line 151, in emit
    *args, **kwargs)
  File "/usr/lib/pitivi/python/pitivi/signalinterface.py", line 126, in emit
    res = cb(*ar, **kw)
  File "/usr/lib/pitivi/python/pitivi/ui/clipproperties.py", line 612, in _selectionChangedCb
    self.zoom_scale.set_value(1.0)
AttributeError: 'TransformationProperties' object has no attribute 'zoom_scale'

A quick fix is attached as a patch that comments line 162 in /usr/lib/pitivi/python/pitivi/ui/clipproperties.py that tried to reset the zoom to 1. The patch does not adversely affect the program in any other way.

--- /usr/lib/pitivi/python/pitivi/ui/clipproperties.py 2013-01-15 22:48:51.230618469 +0530
+++ clipproperties.py 2013-01-16 01:27:13.362470110 +0530
@@ -609,7 +609,7 @@
         else:
             if self._current_tl_obj:
                 self._current_tl_obj = None
- self.zoom_scale.set_value(1.0)
+ #self.zoom_scale.set_value(1.0)
                 self.app.gui.viewer.pipeline.flushSeekVideo()
             self.effect = None
             self.set_sensitive(False)

Tags: patch
Revision history for this message
Experimenter (experimenter) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "patch to comment the reset of zoom to 1" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Benjamin Kerensa (bkerensa) wrote :

Experimenter,

Thanks for the patch I was testing it out and there were some errors with self.effect = None although it did ungroup the clips but for some reason that portion of your patch appears to cause some instability and requires a force quit.

Have you had the same results with your patch?

Revision history for this message
Experimenter (experimenter) wrote : Re: [Bug 1100007] Re: unable to separate video and audio by using "ungroup clips" button

I have not experienced any problems with the patch.
There is only 1 change in the patch @ line 162 in
/usr/lib/pitivi/python/pitivi/ui/clipproperties.py

   - self.zoom_scale.set_value(1.0)
   + #self.zoom_scale.set_value(1.0)

kindly try adding the change manually.
also verify if you are using the ver 0.15.2-0ubuntu0.1 in 12.04 LTS

Regards
Experimenter

On Wednesday 16 January 2013 03:59 PM, Benjamin Kerensa wrote:
> Experimenter,
>
> Thanks for the patch I was testing it out and there were some errors
> with self.effect = None although it did ungroup the clips but for some
> reason that portion of your patch appears to cause some instability and
> requires a force quit.
>
> Have you had the same results with your patch?
>

Revision history for this message
Experimenter (experimenter) wrote :

Excuse me for the mistake.. The patch is at line no 609 and not 162 of
the file
change in the patch @ line 609 in
/usr/lib/pitivi/python/pitivi/ui/clipproperties.py

   - self.zoom_scale.set_value(1.0)
+ #self.zoom_scale.set_value(1.0)

I tried the patch on a different machine and it is working fine there too.

Regards
Experimenter

On Wednesday 16 January 2013 06:00 PM, Experimenter wrote:
> I have not experienced any problems with the patch.
> There is only 1 change in the patch @ line 162 in
> /usr/lib/pitivi/python/pitivi/ui/clipproperties.py
>
> - self.zoom_scale.set_value(1.0)
> + #self.zoom_scale.set_value(1.0)
>
> kindly try adding the change manually.
> also verify if you are using the ver 0.15.2-0ubuntu0.1 in 12.04 LTS
>
> Regards
> Experimenter
>
> On Wednesday 16 January 2013 03:59 PM, Benjamin Kerensa wrote:
>> Experimenter,
>>
>> Thanks for the patch I was testing it out and there were some errors
>> with self.effect = None although it did ungroup the clips but for some
>> reason that portion of your patch appears to cause some instability and
>> requires a force quit.
>>
>> Have you had the same results with your patch?
>>
>

Revision history for this message
Experimenter (experimenter) wrote :

The updated patch should solve your problem. I realized that zoom
setting might be important for some effects. The new patch just catches
the exception if it occurs and the program flow resumes.
please remove the previous patch before applying this.

--- /usr/lib/pitivi/python/pitivi/ui/clipproperties.py
+++ clipproperties.py
@@ -609,7 +609,10 @@
          else:
              if self._current_tl_obj:
                  self._current_tl_obj = None
- self.zoom_scale.set_value(1.0)
+ try:
+ self.zoom_scale.set_value(1.0)
+ except:
+ pass
                  self.app.gui.viewer.pipeline.flushSeekVideo()
              self.effect = None
              self.set_sensitive(False)

Regards
Experimenter

On Wednesday 16 January 2013 10:32 PM, Experimenter wrote:
> Excuse me for the mistake.. The patch is at line no 609 and not 162 of
> the file
> change in the patch @ line 609 in
> /usr/lib/pitivi/python/pitivi/ui/clipproperties.py
>
> - self.zoom_scale.set_value(1.0)
> + #self.zoom_scale.set_value(1.0)
>
> I tried the patch on a different machine and it is working fine there
> too.
>
> Regards
> Experimenter
>
> On Wednesday 16 January 2013 06:00 PM, Experimenter wrote:
>> I have not experienced any problems with the patch.
>> There is only 1 change in the patch @ line 162 in
>> /usr/lib/pitivi/python/pitivi/ui/clipproperties.py
>>
>> - self.zoom_scale.set_value(1.0)
>> + #self.zoom_scale.set_value(1.0)
>>
>> kindly try adding the change manually.
>> also verify if you are using the ver 0.15.2-0ubuntu0.1 in 12.04 LTS
>>
>> Regards
>> Experimenter
>>
>> On Wednesday 16 January 2013 03:59 PM, Benjamin Kerensa wrote:
>>> Experimenter,
>>>
>>> Thanks for the patch I was testing it out and there were some errors
>>> with self.effect = None although it did ungroup the clips but for some
>>> reason that portion of your patch appears to cause some instability and
>>> requires a force quit.
>>>
>>> Have you had the same results with your patch?
>>>
>>
>

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in pitivi (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.