From: C. V. <cri...@gm...> - 2008-10-23 14:35:47
|
Hi! I was compiling VXL (again) and I noticed that the files contrib/brl/bbas/vidl2/vidl2_ffmpeg_ostream_v[1,2].txx use the field AVCodecContext.rtp_mode, from FFmpeg. This field was deprecated for some time, and since last year it's beem removed from the repository code. I'm aware of the discussion about the FFmpeg continuous API changes, but the rtp_mode field it's not even used in VXL! I did a grep on all the files, and only those two files mention rtp_mode, and they only set its value to 1. The field is deprecated and no file is really using it, so why keep these lines? video_enc->rtp_mode= 1; The downside of having them is because VXL may not compile, if the FFmpeg version is [not so] recent. See ya! -- Crístian Deives dos Santos Viana [aka CD1] Google Talk: cri...@gm... |
From: Matthew L. <mat...@gm...> - 2008-10-23 15:23:52
|
Cristian, Currently vidl2 works only with very old versions of FFMPEG. I think the most recent version known to work comes from svn -r11322, which is from about a year ago. I haven't had time to update it, and don't foresee any available time in the near future. Do you know what svn version you have? I think Amitha Perera is working on bringing a copy of FFMPEG into v3p so that we will always have a version that works with VXL available. The plan is to have a stable version in v3p and also support a "head" version that we will try to keep up to date with current FFMPEG, at least to the extent that we have time for. However, this won't happen any time soon. The vidl2_ffmpeg_ostream_params structure was just a dump of all the parameters found in FFMPEG (probably a few years ago). I'm not an expert and I don't know what most of them do. The idea was that anyone who does know what they do can set them and get the full functionality of FFMPEG. Most of these parameters are not used by anyone that I know of. What is the rtp_mode for? Would it be critical for anyone running an old version of FFMPEG? Is this the only thing preventing the code from compiling with a newer version of FFMPEG? If removing rtp_mode will make it compile, and nobody sees a reason it might be needed in old FFMPEG versions, then I will just take it out. --Matt On Oct 23, 2008, at 10:35 AM, Crístian Viana wrote: > Hi! > > I was compiling VXL (again) and I noticed that the files contrib/brl/ > bbas/vidl2/vidl2_ffmpeg_ostream_v[1,2].txx use the field > AVCodecContext.rtp_mode, from FFmpeg. This field was deprecated for > some time, and since last year it's beem removed from the repository > code. I'm aware of the discussion about the FFmpeg continuous API > changes, but the rtp_mode field it's not even used in VXL! I did a > grep on all the files, and only those two files mention rtp_mode, > and they only set its value to 1. The field is deprecated and no > file is really using it, so why keep these lines? > > video_enc->rtp_mode= 1; > > The downside of having them is because VXL may not compile, if the > FFmpeg version is [not so] recent. > > See ya! > > -- > Crístian Deives dos Santos Viana [aka CD1] > Google Talk: cri...@gm... > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's > challenge > Build the coolest Linux based applications with Moblin SDK & win > great prizes > Grand prize is a trip for two to an Open Source event anywhere in > the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/_______________________________________________ > Vxl-maintainers mailing list > Vxl...@li... > https://lists.sourceforge.net/lists/listinfo/vxl-maintainers |
From: C. V. <cri...@gm...> - 2008-10-23 19:17:49
|
Hi Matthew, I was using the latest revision, but I looked at the changelogs and I found out that they removed rtp_mode on February last year. And I don't know ow useful is this rtp_mode field, I'm only talking about it because it stopped my compilation. Now I downgraded FFmpeg to revision 11322 and FFmpeg/libswscale to revision 25485 and VXL compiled fine. The revision number was a nice information for me, maybe it could be written in some README, or INSTALL. The current FFmpeg revision breaks VXL in several places (the function img_convert still exists, but it was removed from avcodec.h; some constants, like CODEC_FLAG_H263P_AIC, don't exist anymore; and so on...). It's hard to guess what's the "right" revision to use. Whenever I use a new computer and I have to download and compile FFmpeg and VXL, I face these problems :( But from now on I'll checkout only this specific revision of FFmpeg (while VXL doesn't package it into v3p). I hope I won't disturb you anymore about FFmpeg. Thanks! -- Crístian Deives dos Santos Viana [aka CD1] Google Talk: cri...@gm... |
From: Antonio G. C. <aga...@de...> - 2008-10-24 12:22:12
|
Hi, For somebody who has developer access to the repository (I can't, yet) I am compiling vxl 1-11.0 on Mandriva 2009 (g++ 4.3.2). Just a detail not included in currect cvs version: Warning at /home/agarrido/investigacion/vxl-1.11.0/contrib/brl/bbas/brdb/brdb_query.h Line 35: typedef ignored I suppose "typedef" could be deleted to avoid this warning. thanks Antonio. |
From: Matthew L. <mat...@gm...> - 2008-10-24 13:31:48
|
Antonio, Thanks, it has been fixed in CVS. --Matt On Oct 24, 2008, at 8:04 AM, Antonio Garrido Carrillo wrote: > > Hi, > For somebody who has developer access to the repository (I can't, yet) > I am compiling vxl 1-11.0 on Mandriva 2009 (g++ 4.3.2). Just a detail > not included in currect cvs version: > > Warning at > /home/agarrido/investigacion/vxl-1.11.0/contrib/brl/bbas/brdb/ > brdb_query.h > Line 35: typedef ignored > > I suppose "typedef" could be deleted to avoid this warning. > > > > thanks > Antonio. > > > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's > challenge > Build the coolest Linux based applications with Moblin SDK & win > great prizes > Grand prize is a trip for two to an Open Source event anywhere in > the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ > Vxl-maintainers mailing list > Vxl...@li... > https://lists.sourceforge.net/lists/listinfo/vxl-maintainers |
From: Antonio G. C. <aga...@de...> - 2008-11-05 17:16:42
|
Hi, After some delay (sorry Matt, too busy), I have added Video4linux2 support to vidl2 The code is not completely finished (no controls for brightness, contrast and so on.), but it's sufficient for many applications. If someone uses Linux and wants to try it, it is in the repository. Of course, suggestions and comments are welcome Antonio. |