Page MenuHome

Patrice Bertrand (PatB)
User

Projects

User Details

User Since
Feb 6 2014, 10:25 AM (258 w, 2 d)

Recent Activity

Sat, Jan 12

Patrice Bertrand (PatB) added a comment to T41748: Remove-Doubles (modifier).

I'm the one who started this thread, and as @Rainer Trummer (aliasguru) says, I am not a blender developper anymore. Back in 2014, it seemed any work on new modifiers was prohibited because a change of paradigm towards node-based modifiers was envisaged. I had built a 'randomizer' modifier that was trashed for that reason.

Sat, Jan 12, 2:44 PM · Mesh Modeling, BF Blender

Aug 19 2015

Patrice Bertrand (PatB) added a comment to T39367: Multi-threading of modifiers.

@Aaron Carlisle (Blendify), it seems there is an ongoing debate as to which multi threading technology is best for Blender and until it is settled little work is undertaken in this field. Yet, with cpu improvements now beeing almost entirely focussed on getting more cores packed, and considering many operations in Blender come down to applying the same algorithm to millions of vertices, it does seem to be a huge performance possibility, as indeed simple experiments on modifiers have shown. But I have learned that working on anything that is not 100% approved is likely to be a waste of time, so it seems to me the first step will be for Blender to have an official choice in multi threading technology.

Aug 19 2015, 5:06 PM · Modifiers, BF Blender
mathieu menuet (bliblubli) awarded T39367: Multi-threading of modifiers a Like token.
Aug 19 2015, 12:54 PM · Modifiers, BF Blender

Jan 13 2015

Patrice Bertrand (PatB) added a comment to D320: Proposed Noise Modifier.

It's ok, I did get the re-engineering of the array modifier into the 2.72, so I will have left a tiny little trace into Blender source code.

Jan 13 2015, 9:43 AM

Jan 12 2015

Patrice Bertrand (PatB) added a comment to D320: Proposed Noise Modifier.

Hello Shane,

Jan 12 2015, 12:11 PM

Dec 22 2014

Patrice Bertrand (PatB) added a comment to D320: Proposed Noise Modifier.

I don't think working on loose parts was the actual blocking factor. Determining loose parts is actually pretty fast, and I had implemented caching of loose parts, so it was not a performance issue. The very notion of loose parts may have been a workaround for finding the original object (in most cases) after array modifiers have been applied. But actually loose parts do have a topological meaning of their own, and so it is not strange or dirty to have something act on loose parts. Many people had suggested that other modifiers could benefit from some loose parts setting.

Dec 22 2014, 10:19 AM

Dec 11 2014

Patrice Bertrand (PatB) awarded D320: Proposed Noise Modifier a Love token.
Dec 11 2014, 10:01 PM

Nov 5 2014

Patrice Bertrand (PatB) added a revision to T42486: Crash on opening file: D879: Correction to T42486.
Nov 5 2014, 9:35 PM · BF Blender
Patrice Bertrand (PatB) added a task to D879: Correction to T42486: T42486: Crash on opening file.
Nov 5 2014, 9:35 PM
Patrice Bertrand (PatB) retitled D879: Correction to T42486 from to Correction to T42486.
Nov 5 2014, 9:33 PM
Patrice Bertrand (PatB) added a comment to T42486: Crash on opening file.

Got it !

Nov 5 2014, 8:54 PM · BF Blender
Patrice Bertrand (PatB) added a comment to T42486: Crash on opening file.

After further investigations, here's where I am:

Nov 5 2014, 8:13 PM · BF Blender
Patrice Bertrand (PatB) added a comment to T42486: Crash on opening file.

The issue relates to array caps with merge option, caps having a subdivision surface active.

Nov 5 2014, 5:05 PM · BF Blender
Patrice Bertrand (PatB) added a comment to T42486: Crash on opening file.

Although indeed
https://developer.blender.org/diffusion/B/browse/master/source/blender/blenkernel/intern/cdderivedmesh.c$2970
is in new implementation. So maybe the "broken" indication is wrong ?

Nov 5 2014, 12:41 PM · BF Blender
Patrice Bertrand (PatB) added a comment to T42486: Crash on opening file.

I will look at it.

Nov 5 2014, 12:31 PM · BF Blender

Oct 2 2014

Patrice Bertrand (PatB) added a comment to T41748: Remove-Doubles (modifier).

If there was a "remove double" modifier, then it would be simple to add settings based on vertex groups. (1) limit the merge to a vertex group, but also (2), limit with given source group onto given target group.
I believe in lightweight modifiers with limited features, used as legos. If and when some node based modifier scheme is introduced, then we would probably use more atomic feature nodes assembled at will. When this comes, it is likely the current modifiers will not be trashed, but simply ported to nodes. In that perspective, it does make sense to introduce such light or atomic modifiers, later to be turned into nodes.

Oct 2 2014, 7:48 PM · Mesh Modeling, BF Blender

Sep 23 2014

Patrice Bertrand (PatB) added a comment to T41914: Animation is different depending on rendered frame range.

Thank you. I will try the workaround.
Just so I can subscribe and follow up, could you please give me the bug reference ?

Sep 23 2014, 1:14 PM · Animation
Patrice Bertrand (PatB) created T41914: Animation is different depending on rendered frame range.
Sep 23 2014, 12:00 PM · Animation

Sep 9 2014

Patrice Bertrand (PatB) added a comment to T41748: Remove-Doubles (modifier).

It's interresting to see, in these examples that the effect of the "remove doubles" may be usefull not only after (after in terms of time not of stack order) other modifiers have acted, but also before, that is to say mesh parts are glued and merged to start with, then because of various modifiers they are torn appart, double vertices get separated and the "double modifier" ceases to operate.

Sep 9 2014, 2:06 PM · Mesh Modeling, BF Blender

Sep 8 2014

Patrice Bertrand (PatB) added a comment to T41748: Remove-Doubles (modifier).
Sep 8 2014, 10:41 PM · Mesh Modeling, BF Blender

Sep 6 2014

Patrice Bertrand (PatB) added a comment to D443: A new implementation of the Array Modifier, without BMesh, gives great performance improvement.

Note that although we call it "remove doubles", (as in "problem"), the operation is really "remove near vertices". In that sense, even though few modifiers would actually create doubles, many could generate very close vertices. The array modifier would do this simply if using scaling (through a scaled empty object), then each instance gets smaller and smaller, and vertices get nearer. Shrinkwrap modifier could also quite often yield vertices that are very close to each other and could be simplified by a "remove doubles".

Sep 6 2014, 12:26 PM

Sep 5 2014

Patrice Bertrand (PatB) added a comment to D443: A new implementation of the Array Modifier, without BMesh, gives great performance improvement.

@Campbell Barton (campbellbarton): do you support the idea of a dedicated "merge doubles" modifier. It would fit in the general idea that what can be done as an non-reversible operation should also be possible to achieve through a modifier. If OK, I can deliver this within a week or two.

Sep 5 2014, 10:23 AM

Aug 13 2014

Patrice Bertrand (PatB) added a comment to D443: A new implementation of the Array Modifier, without BMesh, gives great performance improvement.

Thanks a lot for your feedback @Thomas Beck (plasmasolutions), this is really a heartwarming reward.

Aug 13 2014, 9:43 AM

Jul 29 2014

Patrice Bertrand (PatB) added a comment to T34272: No shadow pass on Cycles from emission or environment lights.

@Matt Outlaw (outlaw3d): note that the suggested compositing is not using the "shadow pass", just the regular image output. Subtracting image with shadows from image without shadows (actually without the objects that cast the shadows) has the effect that everything that was unchanged by shadows is cancelled out, leaving the shadows isolated.
This assumes that you can distinguish between the shadow-casting objects and the shadow-receiving objects, using separate layers. It is definitely just a workaround, not a long term solution.

Jul 29 2014, 10:26 PM · BF Blender, Cycles
Patrice Bertrand (PatB) added a comment to T34272: No shadow pass on Cycles from emission or environment lights.

It seems to me that one could get the same result through compositing, having one layer "WithShadows", another layer "WithoutShadows" where the shadow-casting object does not belong, then using a subtract node in the compositor to isolate what the shadows have altered. Isn't it ?

Jul 29 2014, 8:31 PM · BF Blender, Cycles
Patrice Bertrand (PatB) added a comment to D320: Proposed Noise Modifier.

The nodal modifier design seems to be the holy grail. And yet it could take years to come, and we should not, in the meantime, give up on interresting features just because they would fit so much better with nodes. I think new features can be added within the current model when relevant, and may be then replaced by their nodal equivalent in the future.

Jul 29 2014, 12:53 PM

Jul 28 2014

Patrice Bertrand (PatB) added a comment to D320: Proposed Noise Modifier.

@Thomas Beck (plasmasolutions), @Bastien Montagne (mont29): Hello Thomas. I would love to see it in 2.72 as well. Last time I worked on it was to merge it into the displace modifier. In April, the revision was commandeered by Bastien. He did some cleanup and it seemed ready for release. Maybe Bastien can tell if it's possible to get it into 2.72..

Jul 28 2014, 9:31 AM

May 21 2014

Patrice Bertrand (PatB) updated subscribers of T40297: Crash while ripping an edge when autosmooth is activated..
May 21 2014, 7:14 PM · BF Blender, Mesh Modeling

May 20 2014

Patrice Bertrand (PatB) added a comment to T39566: A new implementation of the Array Modifier, without BMesh, gives 100 fold performance improvement.

Hi,
For those who may be interrested, I'd like to say a few words on the merge doubles algorithm based on the sum of x, y, z coordinates. I picked it up from the remove doubles operator, and tried to improve it very slightly. But this algorithm is actually very poor. When I first saw it, I thought "How clever, the way it manages to handle all 3 coordinates into a single dimention array, which will be sorted, then processed in order". But this transformation of a 3 dimension complex problem into a 1 dimension simple problem is really a complete illusion: it brings very little gain as compared to an algorithm that would simply process vertices based on just their X coordinate, and for each x, would scan all vertices with X within [x-d; x+d]. Think of it this way, in 2 dimensions for a start : imagine all vertices are random points within a square of size DxD. Algorithm A1 would sort by x coordinate only, and compare all vertices within [x-d; x+d], scanning x for all source vertices. Algorithm A2 sorts by (x+y) and compares all vertices such that s=x+y is within [s-2d; s+2d], or best [x-sqrt(2)d; x+sqrt(2)d]. The average number of vertices to be scanned is, in both cases N x d x D. The only difference is that instead of scanning vertical strips, we are scanning diagonal strips around the line of equation y= x0 - x .

May 20 2014, 4:44 PM · Modifiers, BF Blender

May 19 2014

derek barker (lordodin) awarded D443: A new implementation of the Array Modifier, without BMesh, gives great performance improvement a Love token.
May 19 2014, 5:09 PM

May 6 2014

Patrice Bertrand (PatB) added a comment to T39885: Cycles render different between GPU and CPU on transparent shader.

Thank you for your analysis. Still it seems strange to call this anything else than bug. Particularly since it worked fine before. I do agree it is not a major problem, it is not very often that cpu and gpu are used together and must match. I think using shrinkwrap with a very small offset is pretty commonplace.

May 6 2014, 6:25 AM · BF Blender, Cycles

Apr 29 2014

codemanx awarded D443: A new implementation of the Array Modifier, without BMesh, gives great performance improvement a Yellow Medal token.
Apr 29 2014, 3:36 PM

Apr 25 2014

Patrice Bertrand (PatB) added a comment to T39885: Cycles render different between GPU and CPU on transparent shader.

Not exactly a duplicate, since T39761 states that cycles render in view port is not right, while it is correct in a real render. In my case, it is the full render that is different.

Apr 25 2014, 10:07 AM · BF Blender, Cycles

Apr 24 2014

Patrice Bertrand (PatB) added a comment to T39885: Cycles render different between GPU and CPU on transparent shader.

I tested the file on a 2.68RC1, and the 2 renders are identical, so something happened between 2.68 and 2.70, I don't have many versions at hand to try.
My specs: Windows 64bits, GeForce GTX560Ti.

Apr 24 2014, 10:56 PM · BF Blender, Cycles
Patrice Bertrand (PatB) raised the priority of T39885: Cycles render different between GPU and CPU on transparent shader from to Needs Triage.
Apr 24 2014, 10:49 PM · BF Blender, Cycles

Apr 23 2014

Patrice Bertrand (PatB) raised the priority of T39870: Mirror disabled in view mode impacts render mode from to Needs Triage.
Apr 23 2014, 11:18 PM · Modifiers, BF Blender

Apr 11 2014

Patrice Bertrand (PatB) added a comment to T39010: Array modifier is slow when "first last" is enabled.

version 2.67 gives same results.

Apr 11 2014, 2:39 PM · BF Blender
Patrice Bertrand (PatB) added a comment to T39010: Array modifier is slow when "first last" is enabled.

Hello,
I ran some tests. Taking a Suzanne with subsurf at 2, thus 7958 verts each item. Another identical Suzanne is used for caping.

Apr 11 2014, 2:35 PM · BF Blender

Apr 5 2014

Patrice Bertrand (PatB) updated subscribers of T39608: Blender 2.70 crashes when performing union.
Apr 5 2014, 10:38 PM · BF Blender

Apr 4 2014

derek barker (lordodin) awarded T39566: A new implementation of the Array Modifier, without BMesh, gives 100 fold performance improvement a Like token.
Apr 4 2014, 6:32 AM · Modifiers, BF Blender

Apr 3 2014

Patrice Bertrand (PatB) updated the task description for T39566: A new implementation of the Array Modifier, without BMesh, gives 100 fold performance improvement.
Apr 3 2014, 10:57 AM · Modifiers, BF Blender
Patrice Bertrand (PatB) updated the task description for T39566: A new implementation of the Array Modifier, without BMesh, gives 100 fold performance improvement.
Apr 3 2014, 10:40 AM · Modifiers, BF Blender

Apr 2 2014

Patrice Bertrand (PatB) updated subscribers of T39567: Simple Deform modifier error.
Apr 2 2014, 10:49 PM · Modifiers, BF Blender
Patrice Bertrand (PatB) claimed T39566: A new implementation of the Array Modifier, without BMesh, gives 100 fold performance improvement.
Apr 2 2014, 9:55 PM · Modifiers, BF Blender

Mar 30 2014

Patrice Bertrand (PatB) updated subscribers of T38551: Minor code cleanups for Visual Studio 2013, which now has a better standard C++ library.
Mar 30 2014, 3:04 PM · BF Blender, Platform: Windows

Mar 29 2014

Patrice Bertrand (PatB) updated subscribers of T39010: Array modifier is slow when "first last" is enabled.
Mar 29 2014, 9:24 AM · BF Blender

Mar 28 2014

Patrice Bertrand (PatB) added a comment to T39451: Add Vector tests for bl_pyapi_mathutils.py.

Hello,
I created diff D433 with 20 something tests for Vector functions.

Mar 28 2014, 9:55 AM · BF Blender, Restricted Project

Mar 27 2014

Patrice Bertrand (PatB) added a comment to T39451: Add Vector tests for bl_pyapi_mathutils.py.

I'll take this one.

Mar 27 2014, 9:28 AM · BF Blender, Restricted Project

Mar 23 2014

Patrice Bertrand (PatB) raised the priority of T39367: Multi-threading of modifiers from to Needs Triage.
Mar 23 2014, 6:10 PM · BF Blender, Modifiers

Mar 21 2014

Johannes Thomas Meyer (Samoth) awarded D320: Proposed Noise Modifier a Love token.
Mar 21 2014, 3:08 PM

Mar 18 2014

Patrice Bertrand (PatB) updated subscribers of T39247: 2.70 CUDA Error: Not found in cuModuleGetTextRef(&texref,cuModule, name).
Mar 18 2014, 10:37 PM · BF Blender

Mar 11 2014

tommy ob3lisk (ob3lisk) awarded D320: Proposed Noise Modifier a Love token.
Mar 11 2014, 9:40 PM

Mar 10 2014

Bartosz Moniewski (monio) awarded D320: Proposed Noise Modifier a Love token.
Mar 10 2014, 8:13 PM

Mar 8 2014

Patrice Bertrand (PatB) added a comment to T38913: Game engine crash with Arraymod + cap-object.

Hello,
I gave it a try, and have pinpointed the bug, but only partially solved. Tried to upload the patch, but ended up with a new Task. Sorry, I don't master the process. See comments associated with task T39056. I copy hereunder:

Mar 8 2014, 10:23 PM · Game Engine, Modifiers
Patrice Bertrand (PatB) raised the priority of T39056: Pinpointing crash related to T38913 and incomplete fix: crash avoided but array caps still not accounted for in collisions. from to Needs Triage.
Mar 8 2014, 10:17 PM · Game Engine

Mar 7 2014

Patrice Bertrand (PatB) updated subscribers of T39004: Laplacian Deform incorrect with > 10K faces.
Mar 7 2014, 4:46 PM · Modifiers, BF Blender
Patrice Bertrand (PatB) updated subscribers of T38317: Randomize (vertices).
Mar 7 2014, 3:47 PM · Quick Hacks

Mar 6 2014

mathieu menuet (bliblubli) awarded D320: Proposed Noise Modifier a Love token.
Mar 6 2014, 8:46 AM

Feb 17 2014

Karja Krähwald (karja) awarded D320: Proposed Noise Modifier a Love token.
Feb 17 2014, 12:24 AM

Feb 15 2014

Patrice Bertrand (PatB) closed T38527: Randomness in arrays - first attempt as Resolved.
Feb 15 2014, 9:55 PM · Mesh Modeling
Patrice Bertrand (PatB) added a comment to T38527: Randomness in arrays - first attempt.

I have chosen to implement this as "noise modifier", of which I just submitted the patch. So this is closed.

Feb 15 2014, 9:54 PM · Mesh Modeling

Feb 11 2014

Patrice Bertrand (PatB) added a comment to T38527: Randomness in arrays - first attempt.

Thank you for your feedback.

Feb 11 2014, 10:12 AM · Mesh Modeling

Feb 6 2014

Patrice Bertrand (PatB) updated the task description for T38527: Randomness in arrays - first attempt.
Feb 6 2014, 8:49 PM · Mesh Modeling
Patrice Bertrand (PatB) claimed T38527: Randomness in arrays - first attempt.
Feb 6 2014, 8:48 PM · Mesh Modeling