Re: [Bf-committers] Blender File reader SDK (Dan Eicher)
Thanks a lot Dan, But what are you talking about ? Which C++ code ? Where can I found it or got some information about it ? Thanks Krys ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
[Bf-committers] Improvement of BGE rendering for CAVE systems
Hi, I remind you a patch we would like to integrate inside blender : http://projects.blender.org/tracker/?func=detail&atid=127&aid=29161&group_id=9 I would like to know if somebody can review our patch. Regards Damien Touraine -- Damien TOURAINE - Ingénieur de Recherche CNRS, LIMSI-CNRS Groupe de RV&A "VENISE", (http://www.limsi.fr/venise/) Bat. 508, Universite Paris-Sud 91403 Orsay cedex - +33 1 69 85 81 64 ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [47736] trunk/blender/source/blender/ blenkernel/intern/armature.c: Fix invalid array index in armature_deform_verts().
Hey Nick! I've been thinking about such a fix, but that's not actually full fix. There're still lot's of places in blender who assumes def_nr is not negative. And the real fix would actually be to figure out why this index became negative due to it shouldn't actually. Checked the code, but can't find place where it could happen. On Mon, Jun 11, 2012 at 5:01 PM, Nicholas Bishop wrote: > Revision: 47736 > > http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=47736 > Author: nicholasbishop > Date: 2012-06-11 11:00:58 + (Mon, 11 Jun 2012) > Log Message: > --- > Fix invalid array index in armature_deform_verts(). > > Check that the def_nr is non-negative before using as index. > > Fixes bug [#31700] Crash when opening .blend file on 64bit environment > > Modified Paths: > -- >trunk/blender/source/blender/blenkernel/intern/armature.c > > Modified: trunk/blender/source/blender/blenkernel/intern/armature.c > === > --- trunk/blender/source/blender/blenkernel/intern/armature.c 2012-06-11 > 10:31:24 UTC (rev 47735) > +++ trunk/blender/source/blender/blenkernel/intern/armature.c 2012-06-11 > 11:00:58 UTC (rev 47736) > @@ -986,7 +986,7 @@ > >for (j = dvert->totweight; j != 0; j--, dw++) { >const int index = dw->def_nr; > - if (index < defbase_tot && (pchan = > defnrToPC[index])) { > + if (index >= 0 && index < defbase_tot && > (pchan = defnrToPC[index])) { >float weight = dw->weight; >Bone *bone = pchan->bone; >pdef_info = pdef_info_array + > defnrToPCIndex[index]; > > ___ > Bf-blender-cvs mailing list > bf-blender-...@blender.org > http://lists.blender.org/mailman/listinfo/bf-blender-cvs > -- With best regards, Sergey Sharybin ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] FFmpeg api usage update - need help with testing
Hey Sergey, I tried to compile with your patch and ffmpeg-11 from debian-multimedia.org (7:0.11-dmo2), and these packages feature no more libavcodec/opt.h file. I commented it in writeffmpeg.c and ffmpeg_compat.c files, with no problem so far. Also, libs are now in /usr/lib/x86_64-linux-gnu, so had to add the FFMPEG_LIBPATH path manually in CMake, this may need some fix (perhaps create a find_ffmpeg macro?)… Else didn’t had time to test it seriously yet (only a very short glance), but saw no problem till now! :) Best regards, Bastien Le lun. 04 juin 2012 22:17:35 CEST, Sergey Sharybin a écrit : > Hello, Blender Community! > > It was quite known fact that Blender uses lots of api symbols which are > declared deprecated in FFmpeg. > Since FFmpeg version 0.11 some of them vere removed and Blender needs to be > updated a bit, here > is a patch for this [1]. > > But before commiting wanted to be sure there's no regressions. For example > there's still unresolved stuff > going on with av_update_cur_dts -- it was removed from FFmpeg because it's > not normally needed hack, > but it might be needed for correct proxies/timecode generation. > > So if you've got a free minute for tests please let me know if there're > some regressions or everything is smooth. > > Thanks! > > [1] http://www.pasteall.org/32592/diff > ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [47736] trunk/blender/source/blender/ blenkernel/intern/armature.c: Fix invalid array index in armature_deform_verts().
Makes sense. Do you reckon this is an acceptable hack for this specific instance, or should I revert the commit? -Nicholas On Mon, Jun 11, 2012 at 7:04 AM, Sergey Sharybin wrote: > Hey Nick! > > I've been thinking about such a fix, but that's not actually full fix. > There're still lot's of places in blender who assumes > def_nr is not negative. And the real fix would actually be to figure out > why this index became negative due to it > shouldn't actually. > > Checked the code, but can't find place where it could happen. > > On Mon, Jun 11, 2012 at 5:01 PM, Nicholas Bishop > wrote: > >> Revision: 47736 >> >> http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=47736 >> Author: nicholasbishop >> Date: 2012-06-11 11:00:58 + (Mon, 11 Jun 2012) >> Log Message: >> --- >> Fix invalid array index in armature_deform_verts(). >> >> Check that the def_nr is non-negative before using as index. >> >> Fixes bug [#31700] Crash when opening .blend file on 64bit environment >> >> Modified Paths: >> -- >> trunk/blender/source/blender/blenkernel/intern/armature.c >> >> Modified: trunk/blender/source/blender/blenkernel/intern/armature.c >> === >> --- trunk/blender/source/blender/blenkernel/intern/armature.c 2012-06-11 >> 10:31:24 UTC (rev 47735) >> +++ trunk/blender/source/blender/blenkernel/intern/armature.c 2012-06-11 >> 11:00:58 UTC (rev 47736) >> @@ -986,7 +986,7 @@ >> >> for (j = dvert->totweight; j != 0; j--, dw++) { >> const int index = dw->def_nr; >> - if (index < defbase_tot && (pchan = >> defnrToPC[index])) { >> + if (index >= 0 && index < defbase_tot && >> (pchan = defnrToPC[index])) { >> float weight = dw->weight; >> Bone *bone = pchan->bone; >> pdef_info = pdef_info_array + >> defnrToPCIndex[index]; >> >> ___ >> Bf-blender-cvs mailing list >> bf-blender-...@blender.org >> http://lists.blender.org/mailman/listinfo/bf-blender-cvs >> > > > > -- > With best regards, Sergey Sharybin > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [47736] trunk/blender/source/blender/ blenkernel/intern/armature.c: Fix invalid array index in armature_deform_verts().
Well, personally i wouldn't mind keeping this -- at least it helps open file and likely reconfigure some armatures so they're not buggy again. But not sure about how to fix that issue which lead to such a problem. Probably the only case is to ask users to try to reproduce the issue again. At least that's what we're doing when some meshes are corrupted.. On Mon, Jun 11, 2012 at 5:10 PM, Nicholas Bishop wrote: > Makes sense. Do you reckon this is an acceptable hack for this > specific instance, or should I revert the commit? > > -Nicholas > > On Mon, Jun 11, 2012 at 7:04 AM, Sergey Sharybin > wrote: > > Hey Nick! > > > > I've been thinking about such a fix, but that's not actually full fix. > > There're still lot's of places in blender who assumes > > def_nr is not negative. And the real fix would actually be to figure out > > why this index became negative due to it > > shouldn't actually. > > > > Checked the code, but can't find place where it could happen. > > > > On Mon, Jun 11, 2012 at 5:01 PM, Nicholas Bishop > > wrote: > > > >> Revision: 47736 > >> > >> > http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=47736 > >> Author: nicholasbishop > >> Date: 2012-06-11 11:00:58 + (Mon, 11 Jun 2012) > >> Log Message: > >> --- > >> Fix invalid array index in armature_deform_verts(). > >> > >> Check that the def_nr is non-negative before using as index. > >> > >> Fixes bug [#31700] Crash when opening .blend file on 64bit environment > >> > >> Modified Paths: > >> -- > >>trunk/blender/source/blender/blenkernel/intern/armature.c > >> > >> Modified: trunk/blender/source/blender/blenkernel/intern/armature.c > >> === > >> --- trunk/blender/source/blender/blenkernel/intern/armature.c > 2012-06-11 > >> 10:31:24 UTC (rev 47735) > >> +++ trunk/blender/source/blender/blenkernel/intern/armature.c > 2012-06-11 > >> 11:00:58 UTC (rev 47736) > >> @@ -986,7 +986,7 @@ > >> > >>for (j = dvert->totweight; j != 0; j--, dw++) { > >>const int index = dw->def_nr; > >> - if (index < defbase_tot && (pchan = > >> defnrToPC[index])) { > >> + if (index >= 0 && index < defbase_tot && > >> (pchan = defnrToPC[index])) { > >>float weight = dw->weight; > >>Bone *bone = pchan->bone; > >>pdef_info = pdef_info_array + > >> defnrToPCIndex[index]; > >> > >> ___ > >> Bf-blender-cvs mailing list > >> bf-blender-...@blender.org > >> http://lists.blender.org/mailman/listinfo/bf-blender-cvs > >> > > > > > > > > -- > > With best regards, Sergey Sharybin > > ___ > > Bf-committers mailing list > > Bf-committers@blender.org > > http://lists.blender.org/mailman/listinfo/bf-committers > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers > -- With best regards, Sergey Sharybin ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [47736] trunk/blender/source/blender/ blenkernel/intern/armature.c: Fix invalid array index in armature_deform_verts().
Would prefer to revert this fix, if this is ever negative its going to give errors all over and be unreliable (crash somewhere else). If we find this was possible in previous releases - it should be corrected on file-read for example, but not for modifiers/tools. or - accept that it can happen and then check it everywhere (but think this is overkill). On Mon, Jun 11, 2012 at 1:31 PM, Sergey Sharybin wrote: > Well, personally i wouldn't mind keeping this -- at least it helps open > file and likely reconfigure some > armatures so they're not buggy again. > > But not sure about how to fix that issue which lead to such a problem. > Probably the only case is to ask > users to try to reproduce the issue again. At least that's what we're doing > when some meshes are > corrupted.. > > On Mon, Jun 11, 2012 at 5:10 PM, Nicholas Bishop > wrote: > >> Makes sense. Do you reckon this is an acceptable hack for this >> specific instance, or should I revert the commit? >> >> -Nicholas >> >> On Mon, Jun 11, 2012 at 7:04 AM, Sergey Sharybin >> wrote: >> > Hey Nick! >> > >> > I've been thinking about such a fix, but that's not actually full fix. >> > There're still lot's of places in blender who assumes >> > def_nr is not negative. And the real fix would actually be to figure out >> > why this index became negative due to it >> > shouldn't actually. >> > >> > Checked the code, but can't find place where it could happen. >> > >> > On Mon, Jun 11, 2012 at 5:01 PM, Nicholas Bishop >> > wrote: >> > >> >> Revision: 47736 >> >> >> >> >> http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=47736 >> >> Author: nicholasbishop >> >> Date: 2012-06-11 11:00:58 + (Mon, 11 Jun 2012) >> >> Log Message: >> >> --- >> >> Fix invalid array index in armature_deform_verts(). >> >> >> >> Check that the def_nr is non-negative before using as index. >> >> >> >> Fixes bug [#31700] Crash when opening .blend file on 64bit environment >> >> >> >> Modified Paths: >> >> -- >> >> trunk/blender/source/blender/blenkernel/intern/armature.c >> >> >> >> Modified: trunk/blender/source/blender/blenkernel/intern/armature.c >> >> === >> >> --- trunk/blender/source/blender/blenkernel/intern/armature.c >> 2012-06-11 >> >> 10:31:24 UTC (rev 47735) >> >> +++ trunk/blender/source/blender/blenkernel/intern/armature.c >> 2012-06-11 >> >> 11:00:58 UTC (rev 47736) >> >> @@ -986,7 +986,7 @@ >> >> >> >> for (j = dvert->totweight; j != 0; j--, dw++) { >> >> const int index = dw->def_nr; >> >> - if (index < defbase_tot && (pchan = >> >> defnrToPC[index])) { >> >> + if (index >= 0 && index < defbase_tot && >> >> (pchan = defnrToPC[index])) { >> >> float weight = dw->weight; >> >> Bone *bone = pchan->bone; >> >> pdef_info = pdef_info_array + >> >> defnrToPCIndex[index]; >> >> >> >> ___ >> >> Bf-blender-cvs mailing list >> >> bf-blender-...@blender.org >> >> http://lists.blender.org/mailman/listinfo/bf-blender-cvs >> >> >> > >> > >> > >> > -- >> > With best regards, Sergey Sharybin >> > ___ >> > Bf-committers mailing list >> > Bf-committers@blender.org >> > http://lists.blender.org/mailman/listinfo/bf-committers >> ___ >> Bf-committers mailing list >> Bf-committers@blender.org >> http://lists.blender.org/mailman/listinfo/bf-committers >> > > > > -- > With best regards, Sergey Sharybin > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers -- - Campbell ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] Blender File reader SDK (Dan Eicher)
On Mon, Jun 11, 2012 at 1:04 AM, wrote: > Thanks a lot Dan, > > But what are you talking about ? Which C++ code ? > Where can I found it or got some information about it ? > > Thanks > > Krys > > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers Makesrna generates a *huge* C++ wrapper file (RNA_blender_cpp.h) in the build directory (source/blender/makesrna/intern on cmake, not sure what scons does). It doesn't get installed anywhere (anymore) so you either have to do complex include paths into the build dir or move it (them, you need a couple other headers too) by hand. If you make a custom python module and use the C++ api then everything just works, haven't tried outside python so can't say how that works -- blender doesn't have any other way to dynamically load libs so python is pretty much it these days. Just look in intern/cycles to see how the professionals do it... Dan ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
[Bf-committers] Invalid ArtifactID: tracker: [bf-blender-Game Engine][31787]
Hi, The link to this tracker entry gives an 'Artifact: Invalid ArtifactID'. Did the entry get removed? http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 (also I can't see how this is a BGE bug, but maybe the tracker file shows why) Thanks, - Dalai -- Forwarded message -- From: Date: 2012/6/11 Subject: [bf-blender-Game Engine][31787] OSX cursor shape wrong when passing outside Blender workspace window. To: nore...@projects.blender.org Game Engine item #31787, was changed at 2012-06-11 07:00 by Ton Roosendaal You can respond by visiting: http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 Or by replying to this e-mail entering your response between the following markers: #+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+ (enter your response here, only in plain text format) #+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+ Status: Open Priority: 3 Submitted By: Steve Turnbull (zavigny) Assigned to: Nobody (None) Summary: OSX cursor shape wrong when passing outside Blender workspace window. Category: None Status: New Duplicates: Initial Comment: A very minor bug. The cursor in OSX shows the change from the Arrow to the SplitVCursor correctly when crossing window boundaries, but incorrectly remains the SplitVCursor rather then the Arrow when passing outside of Blenders workspace window. See Attachment. -- Comment By: Ton Roosendaal (ton) Date: 2012-06-11 11:49 Message: Moved from Blender 2.6 Bug Tracker to Game Engine -- You can respond by visiting: http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] Invalid ArtifactID: tracker: [bf-blender-Game Engine][31787]
It's here: http://projects.blender.org/tracker/index.php?func=detail&aid=31787&group_id=9&atid=498 We moved it because of a test (moving permissions in the tracker). It is a normal Blender bug, not GE related. We decided to drop the Game Engine category in the Blender 2.6 tracker, all GE bugs now go into the dedicated GE tracker. Regards, Thomas Am 11.06.2012 20:38, schrieb Dalai Felinto: > Hi, > The link to this tracker entry gives an 'Artifact: Invalid ArtifactID'. > Did the entry get removed? > > http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 > > (also I can't see how this is a BGE bug, but maybe the tracker file shows > why) > > Thanks, > - > Dalai > > -- Forwarded message -- > From: > Date: 2012/6/11 > Subject: [bf-blender-Game Engine][31787] OSX cursor shape wrong when > passing outside Blender workspace window. > To: nore...@projects.blender.org > > > Game Engine item #31787, was changed at 2012-06-11 07:00 by Ton Roosendaal > You can respond by visiting: > http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 > Or by replying to this e-mail entering your response between the following > markers: > #+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+ > (enter your response here, only in plain text format) > #+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+#+ > > Status: Open > Priority: 3 > Submitted By: Steve Turnbull (zavigny) > Assigned to: Nobody (None) > Summary: OSX cursor shape wrong when passing outside Blender workspace > window. > Category: None > Status: New > Duplicates: > > > Initial Comment: > A very minor bug. The cursor in OSX shows the change from the Arrow to the > SplitVCursor correctly when crossing window boundaries, but incorrectly > remains the SplitVCursor rather then the Arrow when passing outside of > Blenders workspace window. See Attachment. > > -- > > Comment By: Ton Roosendaal (ton) > Date: 2012-06-11 11:49 > > Message: > Moved from Blender 2.6 Bug Tracker to Game Engine > > -- > > You can respond by visiting: > http://projects.blender.org/tracker/?func=detail&atid=306&aid=31787&group_id=9 > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers -- Thomas Dinges Blender Developer, Artist and Musician www.dingto.org ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
Re: [Bf-committers] Improvement of BGE rendering for CAVE systems
Hi Damien, I remember the patch, but I also remember that you were gathering more feedback before pushing the final version of the patch. (from the tracker) "for the moment, I will wait for feebacks from users from our lab and other Virtual Environment [...] before modifying the patch and resubmit it to this blender patch page." Is everyone happy with the current status of it? Regards, Dalai 2012/6/11 Damien Touraine > Hi, > > I remind you a patch we would like to integrate inside blender : > > http://projects.blender.org/tracker/?func=detail&atid=127&aid=29161&group_id=9 > > I would like to know if somebody can review our patch. > > Regards > Damien Touraine > > -- > > Damien TOURAINE - Ingénieur de Recherche CNRS, LIMSI-CNRS > Groupe de RV&A "VENISE", (http://www.limsi.fr/venise/) > Bat. 508, Universite Paris-Sud 91403 Orsay cedex - +33 1 69 85 81 64 > > > ___ > Bf-committers mailing list > Bf-committers@blender.org > http://lists.blender.org/mailman/listinfo/bf-committers > ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers
[Bf-committers] changing select threshold!
hi i searched, posted a thread on BA.org to know of away of lowering select threshold i need to do that is because it is very hard to select with many vertices around basically i need to click 3-4 times to get the wanted vertex, any ideas? Regards Yousef Harfoush ba...@msn.com ___ Bf-committers mailing list Bf-committers@blender.org http://lists.blender.org/mailman/listinfo/bf-committers