bugGNU Octave - Bugs: bug #61693, [octave forge] (video) Octave...

 
 

bug #61693: [octave forge] (video) Octave video package can’t change default FrameRate

Submitted by:  Razdvizh <razdvizh>
Submitted on:  Fri 17 Dec 2021 06:08:41 PM UTC  
 
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Feature Request
Status:  Fixed Assigned to:  None
Originator Name:  Razdvizh Open/Closed:  Closed
Release:  dev Operating System:  Any

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

( Jump to the original submission)

Fri 14 Jan 2022 09:49:16 PM UTC, comment #8: 

Thanks for pushing. I tested the new version on real life videos and it now works for me.

Closing report as fixed.

Pantxo Diribarne <pantxo>
Project Member
Fri 24 Dec 2021 10:38:10 AM UTC, comment #7: 

Thanks for reviewing pushing. No, I don't have write access to the video package repo.
As for a new release, why not, if Kai id not too busy.

Pantxo Diribarne <pantxo>
Project Member
Thu 23 Dec 2021 07:39:10 PM UTC, comment #6: 

Ah yes, sorry

Andreas Weber <andy1978>
Project Member
Thu 23 Dec 2021 07:37:24 PM UTC, comment #5: 

@andy1978 do you mean VideoWriter.m?

Razdvizh <razdvizh>
Thu 23 Dec 2021 05:55:08 PM UTC, comment #4: 

Thank you Pantxo,
I've added your patch here: https://sourceforge.net/p/octave/video/ci/2857563002d005f0d07c3ccd9db5ad80fff95155

Do you have access to the video repo? I guess yes so please feel free to apply patches on your will.

Do we need a release yet? @razdvizh: you can just replace the VideoWriter.h in your current installation.

Thank you, Andy

Andreas Weber <andy1978>
Project Member
Thu 23 Dec 2021 12:18:38 PM UTC, comment #3: 

I attached an updated patch without trailing white spaces.

(file #52561)

Pantxo Diribarne <pantxo>
Project Member
Thu 23 Dec 2021 11:55:27 AM UTC, comment #2: 

I attached a patch which seems to do the job. Adding the maintainer of the packege in cc

(file #52558)

Pantxo Diribarne <pantxo>
Project Member
Tue 21 Dec 2021 06:02:40 PM UTC, comment #1: 

Confirmed. It is currently not possible to change the FrameRate property.

Pantxo Diribarne <pantxo>
Project Member
Fri 17 Dec 2021 06:08:41 PM UTC, original submission:  

s=VideoWriter("/tmp/test.mp4")

s =

class VideoWriter:
ColorChannels = 3
Duration [s] = 0.00
FileFormat = avi
Filename = test.mp4
FrameCount = 0
FrameRate [1/s] = 30
Height [px] =
Width [px] =
Path = /tmp
VideoBitsPerPixel = 24
VideoCompressionMethod =
VideoFormat =
FourCC =
FFmpeg_versions = Lavu56.31.100, SwS5.5.100, Lavc58.54.100, Lavf58.29.100

s.FrameRate=60

error: subsasgn: property `FrameRate’ has private access and cannot be set in this context

I would request this property to be accessed and be changeable

Razdvizh <razdvizh>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #52561:  bug61693_2.patch added by pantxo (3KiB - text/x-patch)
file #52558:  bug61693.patch added by pantxo (3KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by pantxo
  • -email is unavailable- added by pantxo (Posted a comment)
  • -email is unavailable- added by razdvizh (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-01-14 pantxo StatusReady For Test => Fixed
        Open/ClosedOpen => Closed
    2021-12-23 andy1978 StatusPatch Submitted => Ready For Test
    2021-12-23 pantxo Attached File- => Added bug61693_2.patch, #52561
    2021-12-23 pantxo Attached File- => Added bug61693.patch, #52558
        StatusConfirmed => Patch Submitted
        Carbon-Copy- => Added andy1978
    2021-12-21 pantxo Item GroupNone => Feature Request
        StatusNone => Confirmed

    Back to the top


    Powered by Savane 3.7