7 years agofix compilation for mingw-w64
Antony Riakiotakis [Mon, 13 Jan 2014 13:46:46 +0000 (15:46 +0200)]
fix compilation for mingw-w64

7 years agoFix cycles texture crash on win x86-64 + msvc 11
Sv. Lockal [Mon, 13 Jan 2014 14:21:34 +0000 (18:21 +0400)]
Fix cycles texture crash on win x86-64 + msvc 11

Use union for __m128 aliasing; while gcc supports no-strict-aliasing attribute, unions are the most common way to deal with __m128 in msvc.

7 years agoFix crash happening on render after recent imbuf PERSISTENT flag
Sergey Sharybin [Mon, 13 Jan 2014 14:25:42 +0000 (20:25 +0600)]
Fix crash happening on render after recent imbuf PERSISTENT flag

7 years agoRe-bundle new Ceres library
Sergey Sharybin [Mon, 13 Jan 2014 13:11:08 +0000 (19:11 +0600)]
Re-bundle new Ceres library

- Fixes some harmless issues (in cases blender doesn't use Ceres yet)
- Fixes some compilation warnings

7 years agoUI Icons: store icons in git as uncompressed pixmaps (D196)
Campbell Barton [Mon, 13 Jan 2014 12:47:33 +0000 (23:47 +1100)]
UI Icons: store icons in git as uncompressed pixmaps (D196)

this allows for updating icons without committing a new PNG each time
(which is inefficient with git). The data files are converted into a
PNG at builds time and used just as they were before.

7 years agoFix T37886: Material does no update when changing keyframes in dopesheet
Sergey Sharybin [Mon, 13 Jan 2014 13:02:23 +0000 (19:02 +0600)]
Fix T37886: Material does no update when changing keyframes in dopesheet

It was a missing fcurve evaluation in scene update function which lead to
materials only being updated on frame change.

Added the same exception as we've got for the scene animation. It only
runs when there're materials tagged for the update, so wouldn't expect
speed regressions or so.

7 years agoFix T38040: Crash after loading big image file in compositor
Sergey Sharybin [Mon, 13 Jan 2014 12:42:40 +0000 (18:42 +0600)]
Fix T38040: Crash after loading big image file in compositor

Issue was caused by cache limitor removing viewer image buffer
from the memory during compositing. Now made it so all viewer
images are persistent in the memory.

This solves the crash mentioned above and also makes it so
render/compo results are never lost.

Further tweaks are possible, but pretty much happy now, at
least no stoppers for work are there.

7 years agoUI Icons: use a python script rather then a shell (for non unix systems)
Campbell Barton [Mon, 13 Jan 2014 12:18:25 +0000 (23:18 +1100)]
UI Icons: use a python script rather then a shell (for non unix systems)

7 years agoCompilation error fix and strict warning silence for previous commits
Sergey Sharybin [Mon, 13 Jan 2014 10:33:18 +0000 (16:33 +0600)]
Compilation error fix and strict warning silence for previous commits

7 years agoRemove direct lattice modifiers calc from the drawing code
Sergey Sharybin [Mon, 13 Jan 2014 10:29:07 +0000 (16:29 +0600)]
Remove direct lattice modifiers calc from the drawing code

Same as mballs and curves it was only to work around DAG stupidness
which hopefully was fixed in one of the previous commits.

From quick tests everything works fine, if something is broken now
poke me to find a proper solution.

7 years agoAvoid memcpy to self when validating UV layer name
Sergey Sharybin [Mon, 13 Jan 2014 10:26:10 +0000 (16:26 +0600)]
Avoid memcpy to self when validating UV layer name

7 years agoRemove direct mball creation from object conversion
Sergey Sharybin [Mon, 13 Jan 2014 10:20:20 +0000 (16:20 +0600)]
Remove direct mball creation from object conversion

Same as previous commit, mball shall be visible before conversion
meaning it is to be evaluated already.

7 years agoRemove direct mball creation from the drawing code
Sergey Sharybin [Mon, 13 Jan 2014 10:18:08 +0000 (16:18 +0600)]
Remove direct mball creation from the drawing code

This is the same case as curves, and really this is now
totally up to DAG, If something fails, poke me to fix the DAG.

7 years agoMake bvhutil safe for multi-threaded usage
Sergey Sharybin [Fri, 10 Jan 2014 11:21:39 +0000 (17:21 +0600)]
Make bvhutil safe for multi-threaded usage

There were couple of reasons why it wasn't safe for usage from
multiple threads.

First of all, it was trying to cache BVH in derived mesh, which
wasn't safe because multiple threads might have requested BVH
tree and simultaneous reading and writing to the cache became a
big headache.

Solved this with RW lock so now access to BVH cache is safe.

Another issue is causes by the fact that it's not guaranteed
DM to have vert/edge/face arrays pre-allocated and when one
was calling functions like getVertDataArray() array could
have been allocated and marked as temporary. This is REALLY
bad, because NO ONE is ever allowed to modify data which
doesn't belong to him. This lead to situations when multiple
threads were using BVH tree and they run into race condition
with this temporary allocated arrays.

Now bvhtree owns allocated arrays and keeps track of them, so
no race condition happens with temporary data stored in the
derived mesh. This solved threading issues and likely wouldn't
introduce noticeable slowdown. Even when DM was keeping track
of this arrays, they were re-allocated on every BVH creation
anyway, because those arrays were temporary and were freed
with dm->release() call.

We might re-consider this a bit and make it so BVH trees are
allocated when DM itself is being allocated based on the DAG
layout, but that i'd consider an optimization and not something
we need to do 1st priority.

Fixes crash happening with 05_4g_track.blend from Mango after
the threaded object update landed to master.

7 years agoFix T38139: Objects which are in cyclic dependency are not updated
Sergey Sharybin [Fri, 10 Jan 2014 09:39:06 +0000 (15:39 +0600)]
Fix T38139: Objects which are in cyclic dependency are not updated

Graph traversal which is based on counting parents which are still
to be updated fails in cases there are cycles in the graph.

If there are cyclic dependencies in the scene all the objects from
the cycles will be updated in a single thread now one by one. This
makes blender behave the same way as it was before multi-threaded
DAG landed to master.

This needed to tweak depsgraph a bit so now dag_check_cycle() sets
is_acyclic field of DAG forest if there are cycles in the graph.

TODO: It might be possible to save some time on evaluation when
      all the tagged objects were updated in multi-threaded DAG

7 years agoFix T38054: High CPU usage with many objects
Sergey Sharybin [Thu, 9 Jan 2014 13:13:26 +0000 (19:13 +0600)]
Fix T38054: High CPU usage with many objects

This is a regression since threaded dependency graph landed to master.
Root of the issue goes to the loads of graph preparation being done
even if there's nothing to be updated.

The idea of this change is to use ID type recalc bits to determine
whether there're objects to be updated. Generally speaking, we now
check object and object data datablocks with DAG_id_type_tagged()
and if there's no such IDs tagged we skip the whole task pool creation
and so,

The only difficult aspect was that in some circumstances it was possible
that there are tagged objects but nothing in ID recalc bit fields.

There were several different circumstances when it was possible:

* When one assigns object->recalc flag directly DAG flush didn't
  set corresponding bits to ID recalc bits. Partially it is fixed
  by making it so flush will set bitfield, but also for object
  types there's no reason to assign recalc flag directly. Using
  generic DAG_id_type_tag works almost the same fast as direct
  assignment, ensures all the bitflags are set properly and for the
  long run it seems it's what we would actually want to.

* DAG_on_visible_update() didn't set recalc bits at all.

* Some areas were checking for object->recalc != 0, however it is was
  possible that object recalc flag contains PSYS_RECALC_CHILD which
  was never cleaned from there.

  No idea why would we need to assign such a flag when enabling
  scene simplification, this is to be investigated separately.

* It is possible that scene_update_post and frame_update_post handlers
  will modify objects. The issue is that DAG_ids_clear_recalc is called
  just after callbacks, which leaves objects with recalc flags but no
  corresponding bit in ID recalc bitfield. This leads to some kind of
  regression when using ID type tag fields to check whether there objects
  to be updated internally comparing threaded DAG with legacy one.

  For now let's have a workaround which will preserve tag for ID_OB
  if there're objects with OB_RECALC_ALL bits. This keeps behavior
  unchanged comparing with 2.69 release.

7 years agoRemove direct displist creation from array modifier
Sergey Sharybin [Thu, 9 Jan 2014 20:10:43 +0000 (02:10 +0600)]
Remove direct displist creation from array modifier

First of all, it was needed to have that set scenes fix which
was done recently so curve is being evaluated properly on file

And last but not least, also needed to tag DAG node to evaluate
path regardless to curve datablock settings so curve length is
always known.

7 years agoRemove direct displist creation from constraints
Sergey Sharybin [Thu, 9 Jan 2014 20:02:59 +0000 (02:02 +0600)]
Remove direct displist creation from constraints

Since recent DAG commit for set scenes in DAG_on_visible_update()
it seems there're no longer issues with missing curve_cache after
file load.

7 years agoRemove direct displist creation from curve deform
Sergey Sharybin [Thu, 9 Jan 2014 19:45:53 +0000 (01:45 +0600)]
Remove direct displist creation from curve deform

This solves threading conflict which happens when having
multiple objects using Curve Deform modifier with the same
curve datablock. This conflict was caused by the fact that
curve_deform_verts() used to temporary override curve's
flags to make it path is there.

Actually, it was setting CU_FOLLOW flag temporary which
was only used where_on_path() (only in terms that this
temporary assignment only affected this function) but it
is now commented out for a while, so no reason to set
this flag temporary, If it's ever to be done, we'll need
to pass flags as an additional function argument.

For the path creation i've extended DegNode structure
which now holds extra bits which indicates what additional
data depending on the graph topology is to be evaluated.

Currently this is only used to indicate that curve needs
path to be evaluated regardless to cu->flag state. This
is so Curve Deform modifier is always happy.

In the future this flag might also be used to indicate
whether bmesh verts are to update (see recent commit to
3-vertex parent crash fix) or to indicate that the object
is the motherball etc.

7 years agoRemove direct displist creation from BKE_vfont_to_curve_ex()
Sergey Sharybin [Thu, 9 Jan 2014 18:03:49 +0000 (00:03 +0600)]
Remove direct displist creation from BKE_vfont_to_curve_ex()

This goes back to ancient era again and such a call isn't
safe for threading and really DAG is to make it sure display
list for dependencies is always there.

7 years agoCode cleanup: no need to check display list elements in texture space match
Sergey Sharybin [Thu, 9 Jan 2014 17:51:24 +0000 (23:51 +0600)]
Code cleanup: no need to check display list elements in texture space match

7 years agoRemove direct displist creation from object conversion
Sergey Sharybin [Thu, 9 Jan 2014 17:11:50 +0000 (23:11 +0600)]
Remove direct displist creation from object conversion

If the object is visible and editable it means there's no
way DAG to fail to create needed display lists.

7 years agoRemove direct displist creation from bevel code
Sergey Sharybin [Thu, 9 Jan 2014 17:02:45 +0000 (23:02 +0600)]
Remove direct displist creation from bevel code

BKE_curve_bevel_make() is only used from object_handle_update()
friends and never called directly. This means if there's no
display list ready for the bevel object it's something wrong
happened with DAG.

In fact, this check goes back to ancient era and from tests
it appears this check is no longer needed.

7 years agoRemove direct displist creation from drawing code
Sergey Sharybin [Thu, 9 Jan 2014 16:42:11 +0000 (22:42 +0600)]
Remove direct displist creation from drawing code

It was some kind of workaround for DAG glitch in 2009
(commit hash 8c5c7ebb0) and according to the comment
was needed to make select outline show immediately.

After some tests it appears DAG behaves almost fine now
(just needed to make it so layer is flushed properly to
the set scene) and no reason to have rather confusing
call in the code.

7 years agoUI: remove create new directory popup for button in file browser header.
Andrew Buttery [Mon, 13 Jan 2014 09:43:15 +0000 (10:43 +0100)]
UI: remove create new directory popup for button in file browser header.

We do keep it when typing an new directory name in the directory text field.

Reviewed By: brecht, billrey

Differential Revision:

7 years agoSelect Random: add option to de-select
Campbell Barton [Mon, 13 Jan 2014 09:36:38 +0000 (20:36 +1100)]
Select Random: add option to de-select

also made metaball operator behave like the others.

Path originally from Walid Shouman, with own edits.

7 years agoFix cycles standalone compile on some systems, CMAKE_DL_LIBS should be enough.
Marcus von Appen [Mon, 13 Jan 2014 09:33:43 +0000 (10:33 +0100)]
Fix cycles standalone compile on some systems, CMAKE_DL_LIBS should be enough.

Reviewed By: brecht

Differential Revision:

7 years agoCode Cleanup: spelling
Campbell Barton [Mon, 13 Jan 2014 03:53:41 +0000 (14:53 +1100)]
Code Cleanup: spelling

7 years agoFix some harmless warnings that mostly appeared on MinGW64
Antony Riakiotakis [Mon, 13 Jan 2014 02:28:51 +0000 (04:28 +0200)]
Fix some harmless warnings that mostly appeared on MinGW64

7 years agoBevel fixes for profiles: better way to calculate.
Howard Trickey [Sun, 12 Jan 2014 19:06:58 +0000 (14:06 -0500)]
Bevel fixes for profiles: better way to calculate.

It is better to keep the profile as it is perpedicular
to the edge, and then project it onto a given plane
at the corners. Also fixed the interpolation to a
different number of segments when the profile is not

7 years agoFix T38124: Grease Pencil Line thickness
Bastien Montagne [Sun, 12 Jan 2014 17:52:14 +0000 (18:52 +0100)]
Fix T38124: Grease Pencil Line thickness

Issue was in GP draw code: line thickness was not initiated properly (and a null one makes OGL draw a unity-width line).

Also tweaked threshold (when to start a new, width-different OGL linestrip), to make it inversaly proportional to thickness
(so that now, it's 0.2 for a thickness of 1, 0.1 for a thickness of 2, etc.), avoids too big steps when using large thickness.

7 years agoUIList tweak: make active item visible when it changes somehow (useful e.g. when...
Bastien Montagne [Sun, 12 Jan 2014 14:56:47 +0000 (15:56 +0100)]
UIList tweak: make active item visible when it changes somehow (useful e.g. when weight-paintings a rigged mesh).

7 years agoFix compilation for OpenCL (and small stype fixes)
Sv. Lockal [Sun, 12 Jan 2014 14:18:43 +0000 (18:18 +0400)]
Fix compilation for OpenCL (and small stype fixes)

7 years agoCycles: SSE for Voronoi textures (targeted for Haswell CPUs)
Sv. Lockal [Sun, 12 Jan 2014 14:14:00 +0000 (18:14 +0400)]
Cycles: SSE for Voronoi textures (targeted for Haswell CPUs)

Gives up to 15% speedup scenes with voronoi-based textures (up to 25% with volumes) on Haswell. The performance change for other CPUs is much smaller: 1-2%.

Reviewed By: brecht

Differential Revision:

7 years agoStyle Cleanup: whitespace
Campbell Barton [Sun, 12 Jan 2014 11:27:55 +0000 (22:27 +1100)]
Style Cleanup: whitespace

7 years agoStyle Cleanup: whitespace
Campbell Barton [Sun, 12 Jan 2014 11:05:24 +0000 (22:05 +1100)]
Style Cleanup: whitespace

7 years agoUI: More consistency for T/N sidebars. Logic Editor Properties are on the right now...
Thomas Dinges [Sun, 12 Jan 2014 00:00:16 +0000 (01:00 +0100)]
UI: More consistency for T/N sidebars. Logic Editor Properties are on the right now as well, and File Browser Bookmark sidebar uses "T" key, as its on the right.

7 years agoFix T38160, N and T sidebars in Image Editor are swapped. Now the Properties are...
Andrew Buttery [Sat, 11 Jan 2014 23:50:43 +0000 (00:50 +0100)]
Fix T38160, N and T sidebars in Image Editor are swapped. Now the Properties are on the right side, which is consistent with other editors.

Differential Revision:

7 years agoUI: Add a "New" button for the particle system, consistent to other areas like materi...
Andrew Buttery [Sat, 11 Jan 2014 23:39:15 +0000 (00:39 +0100)]
UI: Add a "New" button for the particle system, consistent to other areas like materials, world and textures.

Differential Revision:

7 years agoCode Cleanup: in Cycles SSE replace macros with templates, skip unused code with...
Sv. Lockal [Sat, 11 Jan 2014 18:20:03 +0000 (22:20 +0400)]
Code Cleanup: in Cycles SSE replace macros with templates, skip unused code with preprocessor, simplify casts

7 years agoCycles: mix hair minimum width code with SSE intersection code
Sv. Lockal [Sat, 11 Jan 2014 16:47:30 +0000 (20:47 +0400)]
Cycles: mix hair minimum width code with SSE intersection code

Gives 6.5% speedup for hair.blend from testsuite.

Reviewed By: brecht

Differential Revision:

7 years agoCode Cleanup: move MOUSEX/Y to BGE, describe INBETWEEN_MOUSEMOVE
Campbell Barton [Sat, 11 Jan 2014 14:59:17 +0000 (01:59 +1100)]
Code Cleanup: move MOUSEX/Y to BGE, describe INBETWEEN_MOUSEMOVE

7 years agoEvents: support for buttons 6 & 7 (some trackballs have these) X11 only
Campbell Barton [Sat, 11 Jan 2014 13:38:44 +0000 (00:38 +1100)]
Events: support for buttons 6 & 7 (some trackballs have these) X11 only

Patch by Marcus von Appen

Note: this patch makes ISMOUSE accept INBETWEEN_MOUSEMOVE as a mouse
event where before it didnt.

7 years agoEvents: Use INBETWEEN_MOUSEMOVE for inactive windows too.
Campbell Barton [Sat, 11 Jan 2014 13:30:16 +0000 (00:30 +1100)]
Events: Use INBETWEEN_MOUSEMOVE for inactive windows too.

7 years agoGhost: update multitest for changes in BLF
Campbell Barton [Sat, 11 Jan 2014 12:52:36 +0000 (23:52 +1100)]
Ghost: update multitest for changes in BLF

7 years agoAdd tangent space computation/access from RNA (i.e. python).
Bastien Montagne [Sat, 11 Jan 2014 10:31:44 +0000 (11:31 +0100)]
Add tangent space computation/access from RNA (i.e. python).

This simply mimics code used for loopnormals, to enable py scripts to generate and access (temporary)
a tangent 3D vector and bitangent sign for each loop. Together with the split normals, this allow
to recreate a complete tangent space for normal mapping (bitangent = bitangent_sign * cross(normal, tangent)).

Expects all faces to be tri or quads.

Reviewed By: Brecht, campbellbarton

Differential Revision:

7 years agoImage API: add frame argument to gl_load(), gl_touch()
Campbell Barton [Sat, 11 Jan 2014 10:25:20 +0000 (21:25 +1100)]
Image API: add frame argument to gl_load(), gl_touch()

patch D103 by Krantz Geoffroy

7 years agoFix T38150: correct fix this time
Campbell Barton [Sat, 11 Jan 2014 10:03:21 +0000 (21:03 +1100)]
Fix T38150: correct fix this time

also use fixed size lists for list creation.

7 years agoScons: disable player along with game-engine.
Campbell Barton [Sat, 11 Jan 2014 03:54:57 +0000 (14:54 +1100)]
Scons: disable player along with game-engine.

7 years agoFix T38145: python error popup when pressing shift+Z in blender game mode.
Brecht Van Lommel [Fri, 10 Jan 2014 23:59:22 +0000 (00:59 +0100)]
Fix T38145: python error popup when pressing shift+Z in blender game mode.

7 years agoFix T38129: cycles viewport render display with very bright colors turning black.
Brecht Van Lommel [Fri, 10 Jan 2014 23:49:51 +0000 (00:49 +0100)]
Fix T38129: cycles viewport render display with very bright colors turning black.

This happened when exceeding the maximum value representable by half floats.

7 years agoFix T37817: cycles CUDA detection problem on Windows with non-ascii paths.
Brecht Van Lommel [Fri, 10 Jan 2014 23:47:58 +0000 (00:47 +0100)]
Fix T37817: cycles CUDA detection problem on Windows with non-ascii paths.

7 years agoFix T38149: crash adding metaball with zero radius.
Brecht Van Lommel [Fri, 10 Jan 2014 19:52:18 +0000 (20:52 +0100)]
Fix T38149: crash adding metaball with zero radius.

This incorrectly use abs(), that's for integers, not floats.

7 years agoFix clang warning for (harmless) use of uninitialized variable.
Brecht Van Lommel [Fri, 10 Jan 2014 18:43:31 +0000 (19:43 +0100)]
Fix clang warning for (harmless) use of uninitialized variable.

7 years agoFix T38143: pressing F12 key in the text editor on OS X would type an invalid character.
Brecht Van Lommel [Fri, 10 Jan 2014 18:39:35 +0000 (19:39 +0100)]
Fix T38143: pressing F12 key in the text editor on OS X would type an invalid character.

7 years agoFix T38142: socket interface 'type' enums are not initialized. These are
Lukas Tönne [Fri, 10 Jan 2014 15:51:45 +0000 (16:51 +0100)]
Fix T38142: socket interface 'type' enums are not initialized. These are
not really needed anyway, but need to be adjusted to make use of the
socket value copy methods.

7 years agoFix T38150: BMLayerCollection.items/values docs switched
Campbell Barton [Fri, 10 Jan 2014 11:17:33 +0000 (22:17 +1100)]
Fix T38150: BMLayerCollection.items/values docs switched

7 years agoFix for random crash in localized node group freeing while tweaking
Lukas Tönne [Fri, 10 Jan 2014 08:34:10 +0000 (09:34 +0100)]
Fix for random crash in localized node group freeing while tweaking
group default values.

This can happen when using value sliders for node group input values.
The localized copies were setting the "interface_type" runtime pointer
of the original tree to NULL instead of the new tree (which is created
on-the-fly in general). This type is used in RNA update functions
however, the original tree DNA should not be modified there.

7 years agoOpenCollada: Fix error in printf-format (too much arguments).
Bastien Montagne [Thu, 9 Jan 2014 19:34:02 +0000 (20:34 +0100)]
OpenCollada: Fix error in printf-format (too much arguments).

7 years agoCleanup: mostly, prefer array syntax to pointer arithmetic (keep the later only where...
Bastien Montagne [Thu, 9 Jan 2014 19:11:48 +0000 (20:11 +0100)]
Cleanup: mostly, prefer array syntax to pointer arithmetic (keep the later only where it really makes sense).

7 years agoFix T38135: Global start and end point of the curve was set to zero radius & weight...
Bastien Montagne [Thu, 9 Jan 2014 18:45:26 +0000 (19:45 +0100)]
Fix T38135: Global start and end point of the curve was set to zero radius & weight, when using 'linked' option.

This was (more or less) OK with hand-drawn strokes, as the number of points made it nearly unoticable, but broke completely with line and poly strokes!

Did this when I implemented linked curve feature because it was easier! Now, convert code always adds a heading and trailing point to the curve,
to get initial/final zero radius. Adds even more complexity to those functions... :/

7 years agoGrease pencil 'convert to curve' code: some cleanup & refactor.
Bastien Montagne [Thu, 9 Jan 2014 14:23:28 +0000 (15:23 +0100)]
Grease pencil 'convert to curve' code: some cleanup & refactor.

7 years agoCleanup: Remove tabs in empty lines, use bool when possible instead of int, and mark...
Bastien Montagne [Thu, 9 Jan 2014 10:08:17 +0000 (11:08 +0100)]
Cleanup: Remove tabs in empty lines, use bool when possible instead of int, and mark const values/parameters as such.

7 years agoCode cleanup: remove WIP code came from the GSoC branch
Sergey Sharybin [Thu, 9 Jan 2014 19:23:49 +0000 (01:23 +0600)]
Code cleanup: remove WIP code came from the GSoC branch

DAG node tagging was rather an experiment to make derived render working.
However, it ended up in a whole can of worms and need to be re-considered.
It is likely that regular object update tagging and scene update routines
are to be used for this.

Meanwhile no need to keep extra field in dag node. Would save us the whole
byte of the struct which we can use for other purposes meanwhile.

7 years agoFix own regression in font->curve conversion
Sergey Sharybin [Thu, 9 Jan 2014 17:34:52 +0000 (23:34 +0600)]
Fix own regression in font->curve conversion

Issue was introduced in a2bf25e and was caused by
do_makeDispListCurveTypes() no longer placing nurbs
to cu->nurb list.

Such an operation isn't thread-safe and proper solution
would require having granular update. For until them
just make object conversion take care of filling cu->nurb
in with splines from font.

7 years agoFix T38083: submenu arrow is overlayed by menu hotkey
Campbell Barton [Thu, 9 Jan 2014 15:23:55 +0000 (02:23 +1100)]
Fix T38083: submenu arrow is overlayed by menu hotkey

7 years agoFix T38077: Scaling bones in EditMode drawn using envelope display mode scales joint...
Joshua Leung [Thu, 9 Jan 2014 11:44:03 +0000 (00:44 +1300)]
Fix T38077: Scaling bones in EditMode drawn using envelope display mode scales joint radii instead

When trying to scale bones in EditMode and the bones were drawn using envelope display mode,
this resulted in the joint radii (i.e. the inner part of envelopes) being adjusted instead.
It turns out that this was due to an old hack that was put in place back in 2.4x (see the
tracker logs for full details of the problem here).

This commit introduces the following fixes:
1) Removed the old hack. Scaling (S) works normally now.
2) Ctrl-Alt-S (i.e. "Scale Envelopes/BBones") is as-is.
   That is, it is used to adjust the size falloff-region around a bone
   (i.e. the "dist" property)
3) Added Alt-S hotkey in EditMode for armatures for adjusting the radii of bones.
   This change just means that the "TFM_BONE_ENVELOPE" mode is now able to be
   accessed from the UI as a tool on its own right (instead of being accessible
   via the old undocumented hack). This tool adjusts the radii of the bone joints,
   which define the actual full-influence region of the envelopes.

7 years agoFix T38138: incorrect API docs
Campbell Barton [Thu, 9 Jan 2014 10:48:41 +0000 (21:48 +1100)]
Fix T38138: incorrect API docs

7 years agoFix T38000: vertex parent crash due to threading issues
Sergey Sharybin [Thu, 9 Jan 2014 10:19:51 +0000 (16:19 +0600)]
Fix T38000: vertex parent crash due to threading issues

Issue is causes by vertex parent modifies original BMesh from
a multiple threads. Ideally this is to be done as a separate
update callback for mesh datablock, but it's not so much simple
now (would need to do some re-arranges to DAG which might conflict
with the work from Ali or will double amount of work we did).

So for now use simple solution with mutex lock.

Based on the patch from Campbell Barton with some fixes to make
changes really thread-safe.

Differential Revision:

7 years agoCompilation error fix for NetBSD
Sergey Sharybin [Thu, 9 Jan 2014 10:15:24 +0000 (16:15 +0600)]
Compilation error fix for NetBSD

Based on the patch from Joerg Sonnenberger.

7 years agoFix compilation error on NetBSD
Sergey Sharybin [Thu, 9 Jan 2014 10:03:02 +0000 (16:03 +0600)]
Fix compilation error on NetBSD

Original patch is by Jeorg Sonnenberger, thanks!

7 years agoFix remaining sculpt mode crash using multires modifier, similar cause
Antony Riakiotakis [Thu, 9 Jan 2014 02:28:17 +0000 (04:28 +0200)]
Fix remaining sculpt mode crash using multires modifier, similar cause
to previous sculpt fix (OpenGL access from thread with no context
bound). The fact that this has gone unnoticed so far means that people
are dyntopoing like crazy these days.

7 years agoCode Cleanup: style
Campbell Barton [Thu, 9 Jan 2014 00:44:14 +0000 (11:44 +1100)]
Code Cleanup: style

7 years agoFix T38134: missing cycles update when removing world volume scatter shader.
Brecht Van Lommel [Thu, 9 Jan 2014 00:26:43 +0000 (01:26 +0100)]
Fix T38134: missing cycles update when removing world volume scatter shader.

7 years agoFix T38016: setting Object.matrix_world does not take bone parents into account.
Brecht Van Lommel [Wed, 8 Jan 2014 23:37:41 +0000 (00:37 +0100)]
Fix T38016: setting Object.matrix_world does not take bone parents into account.

7 years agoCode cleanup: remove BKE_object_where_is_calc_simul function.
Brecht Van Lommel [Wed, 8 Jan 2014 23:17:18 +0000 (00:17 +0100)]
Code cleanup: remove BKE_object_where_is_calc_simul function.

It doesn't make any sense anymore with the current depsgraph and probably was
not useful for a long time, just a leftover from the pre 2.04 game engine.

7 years agoFix T38062: normal map baking gave randomly values 127 or 128 in flat areas.
Brecht Van Lommel [Wed, 8 Jan 2014 22:36:11 +0000 (23:36 +0100)]
Fix T38062: normal map baking gave randomly values 127 or 128 in flat areas.

Due to float precision issues it was basically random which of the two was used,
now it's slightly biased towards 128, which is the convention for flat colors.
The small difference between 127 and 128 could give problems with sharp glossy
shaders where it would be visible as seams.

7 years agoFix T38134: cycles viewport render not updating when adding a world volume shader.
Brecht Van Lommel [Wed, 8 Jan 2014 22:12:54 +0000 (23:12 +0100)]
Fix T38134: cycles viewport render not updating when adding a world volume shader.

7 years agoFix T38104: mathutils.cell_vector() always returns (0,0,0)
Denis Declara [Wed, 8 Jan 2014 22:03:09 +0000 (23:03 +0100)]
Fix T38104: mathutils.cell_vector() always returns (0,0,0)

Reviewed By: brecht

7 years agoFix crash cancelling command line render to AVI movie file.
Brecht Van Lommel [Wed, 8 Jan 2014 21:28:29 +0000 (22:28 +0100)]
Fix crash cancelling command line render to AVI movie file.

7 years agoFix T38131: node link buttons (like for cycles in material properties) were not
Brecht Van Lommel [Wed, 8 Jan 2014 19:40:37 +0000 (20:40 +0100)]
Fix T38131: node link buttons (like for cycles in material properties) were not
giving correct colors with theme menu colors set to bright values.

Thanks to Denis Declara for finding the cause of this bug.

7 years agoCycles: workaround for noise performance regression in CUDA 5.5
Sv. Lockal [Wed, 8 Jan 2014 18:25:55 +0000 (22:25 +0400)]
Cycles: workaround for noise performance regression in CUDA 5.5

Use manual ternary operation widening in grad(). Without it nvcc 5.5 produces multiple branch splits with very big branches (because of inlining). This solves 19% performance regression for BMW1M-MikePan.blend.

Also remove one redundant instruction in perlin SSE (when h == 12 or h == 14, then h is always >= 4).

Reviewed By: brecht

Differential Revision:

7 years agoFix T38074: sequence rendering gave "No camera" warning for other scenes that
Brecht Van Lommel [Wed, 8 Jan 2014 17:26:42 +0000 (18:26 +0100)]
Fix T38074: sequence rendering gave "No camera" warning for other scenes that
didn't need a camera because they use only compositing nodes.

7 years agoFix T38105: Action Editor refresh is missing when a keyframe is inserted in 3D view
Thomas Beck [Wed, 8 Jan 2014 16:50:25 +0000 (17:50 +0100)]
Fix T38105: Action Editor refresh is missing when a keyframe is inserted in 3D view

 When yiz insert a keyframe (on a location keying set f.e.) in the 3D-view and you got an action editor open then the action datablock is  not changed (displaying only the button "new") until you hover the action editor with the mouse.

 Added a handler in the action_header_area_listener to handle this case

If non-obvious, some technical note about what the cause of the bug was and
how it was solved.

7 years agoFix T38108: blender freeze in remembering operator properties.
Brecht Van Lommel [Wed, 8 Jan 2014 16:48:38 +0000 (17:48 +0100)]
Fix T38108: blender freeze in remembering operator properties.

7 years agoFix for incorrect use of abs() in intersect_sphere_sphere_2d
Campbell Barton [Wed, 8 Jan 2014 16:07:17 +0000 (03:07 +1100)]
Fix for incorrect use of abs() in intersect_sphere_sphere_2d

7 years agoFix issues with float precision in numinput, like 'R 123' who would show additional...
Bastien Montagne [Wed, 8 Jan 2014 16:04:10 +0000 (17:04 +0100)]
Fix issues with float precision in numinput, like 'R 123' who would show additional 'noise' digits.

Expose float precision helper in UI_interface.h API, so that numinput can use this helper as numbuttons already do.

Reviewed By: campbellbarton

Differential Revision:

7 years agoCode Cleanup: style edits to bevel and remove unneeded boundvert_rep_face call
Campbell Barton [Wed, 8 Jan 2014 13:50:32 +0000 (00:50 +1100)]
Code Cleanup: style edits to bevel and remove unneeded boundvert_rep_face call

7 years agoAdd profile control parameter to Bevel.
Howard Trickey [Wed, 8 Jan 2014 12:40:01 +0000 (07:40 -0500)]
Add profile control parameter to Bevel.

Parameter controls concavity / convexity.
    <.25 means: concave inward
    .25 means: straight slanted
    >.25 means: concave outward
    .5 means: circular (the default)
    1 means: straight along original sides
For now, there is a hard lower limit of .15
because more work is needed to get decent
results in the range below that.

The profile is actually a superellipse, and the
parameter is 1/4 of the exponent in the implicit equation
for a superellipse, except at the extreme values of 0 and 1.

7 years agoCycles: Fix World Homogeneous volume setting, had no influence.
Thomas Dinges [Wed, 8 Jan 2014 10:00:06 +0000 (11:00 +0100)]
Cycles: Fix World Homogeneous volume setting, had no influence.

Thanks to "storm" for noticing this!

7 years agoCode Cleanup: de-duplicate text pasting which only used the first line
Campbell Barton [Wed, 8 Jan 2014 06:39:12 +0000 (17:39 +1100)]
Code Cleanup: de-duplicate text pasting which only used the first line

7 years agoText3d: improve error checking when pasting from files
Campbell Barton [Wed, 8 Jan 2014 05:12:25 +0000 (16:12 +1100)]
Text3d: improve error checking when pasting from files

7 years agoFix for RE_engine_begin_result using a NULL pointer
Campbell Barton [Wed, 8 Jan 2014 04:43:49 +0000 (15:43 +1100)]
Fix for RE_engine_begin_result using a NULL pointer

7 years agoSequencer: prevent unknown scopes from crashing
Campbell Barton [Wed, 8 Jan 2014 04:40:55 +0000 (15:40 +1100)]
Sequencer: prevent unknown scopes from crashing

7 years agoFix for inorrect use of BLI_utf8_invalid_strip, add assert to prevent it happening...
Campbell Barton [Wed, 8 Jan 2014 03:46:06 +0000 (14:46 +1100)]
Fix for inorrect use of BLI_utf8_invalid_strip, add assert to prevent it happening again.

7 years agoUV Unwrap: smart uv unwrap would fail still on some very small faces
Campbell Barton [Wed, 8 Jan 2014 03:02:18 +0000 (14:02 +1100)]
UV Unwrap: smart uv unwrap would fail still on some very small faces

7 years agoFix T38112: Rigid Body can't be created for objects with linked mesh
Sergej Reich [Wed, 8 Jan 2014 02:29:12 +0000 (03:29 +0100)]
Fix T38112: Rigid Body can't be created for objects with linked mesh

Make poll function for rigid body add operators less strict.

7 years agoMuch better solution for negate/inverse in numinput.
Bastien Montagne [Tue, 7 Jan 2014 21:23:03 +0000 (22:23 +0100)]
Much better solution for negate/inverse in numinput.

Previous one was way over complicated, and did not worked with units!

7 years agoFix a small glitch with string generated by numinput, they could cut multi-bytes...
Bastien Montagne [Tue, 7 Jan 2014 19:56:45 +0000 (20:56 +0100)]
Fix a small glitch with string generated by numinput, they could cut multi-bytes utf8 chars (like the '°' of angle values), now call BLI_utf8_invalid_strip() to prevent this.

7 years agoFix T38100: laplacian smooth crash
Campbell Barton [Tue, 7 Jan 2014 16:44:00 +0000 (03:44 +1100)]
Fix T38100: laplacian smooth crash