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

 
 

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

Submitter:  Razdvizh <razdvizh>
Submitted:  Fri 17 Dec 2021 06:08:41 PM UTC
   
 
Category:  Octave 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
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group Member
Thu 23 Dec 2021 07:39:10 PM UTC, comment #6: 

Ah yes, sorry

Andreas Weber <andy1978>
Group 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>
Group 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>
Group 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>
Group 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>
Group 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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group 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.13-d3ae.
    Corresponding source code