Better fix for sequencer crash when text strip doesn't have effect data.

This situation happens when a file with a text effect sequencer strip is
loaded in Blender < 2.76 and saved. This destroys the effect data, causing
a crash in Blender ≥ 2.76.

d2f748a222ad19f prevented the crash when opening such a file, but accessing
the strip still caused a crash. This commit fixes that by actually
initialising the invalid strip. Of course this still causes data loss, but
that already happened by opening & overwriting the file in Blender < 2.76.

Cycles: Fix compilation error after recent logging changes

This file uses std::ostream for helper << operators, so need to make sure
corresponding header is included.

Sequencer: Prevent crash when text strip doesn't have effect data.

Code comments regarding last commit (forgotten to add before)

Using an arbitrary face as the source of the UV data is mostly fine, as
vertices on seams will generally map to different parts of the texture
that have the same color.

This is regarding fed853ea782211298c902759ec8cd8e455d8b41d

Fix T51296: UVs not working for hair emitted from vertices

Compositor: Fix compilation error and crash when using defocus search

Was just wondering what the option is doing. Not sure yet, but let's fix
couple of issues here.

Cycles: Pass string by const reference rather than by value

Some of the functions might have been inlined, but others i don't see
how that was possible (don't think virtual functions can be inlined here).

In any case, better be explicitly optimal in the code.

Cycles: Fix compilation error when building without Glog and no C++11

Fix T51834: Active Object and Groups color difference imperceptible

16 was not enough here, -32 seems to give OK results.

Fix T51965: Custom Bone Shape Wireframe setting affects wireframe thickness.

Clearing of custom bones outline's line thickness was not done at proper
point, wireframe drawing never changes line thickness, only solid draw
with outline does...

Fix T51703: Rigid body with delta transform jumps when transforming

When doing any transformation on a rigid body object that has delta
transforms, the object would be offset by the amount of the delta
transform.

Better fix for rigid body not resimulating

Last fix only accounted for direct changes to the RB settings, but
failed for, say, object transformations. This fix accounts for any
change that might invalidate the RB cache.

Fix rigid body not resimulating after cache invalidation

Fix 9cd6b03187b91bb2c267a45eac3cee7738e0e220 introduced a bug that
prevented simulation after a cache invalidation (for instance when
changing a setting after simulating). This fixes that.

Fix T51950: Abnormally long Cycles OpenCL GPU render times with certain panoramic camera settings

The problem here was that when a "invalid" path is generated by the panoramic camera, it was tagged
as RAY_TO_REGENERATE with the intention of generating a new path in kernel_buffer_update.

However, since that state was not handled in kernel_queue_enqueue, kernel_buffer_update did not
process the path which resulted in an infinite loop.

Cycles: Add missing split kernel to CPUDevice

Fix T51523: Lattice modifier affecting particles even when disabled

Particles were ignoring the lattice modifier visibility, and
"influence" property.

Fixed build error on Windows / VS2015

Alembic: fix T51820 for sequence files not loading properly.

Alembic: avoid crashing when doing certain file operations.

Things like missing directories are now properly checked for, rather than
crashing Blender.

This also adds support for relative paths when opening an ABC file.

Alembic: renamed offset → sequence_offset

Fix T51883: Wrong matrix computation in "Make Duplicates Real"

Fix missing return error introduced by last commit

End of non-void function was being reached since
f5535fcb83fd7c1374697923b43565c9e303d225

Fi T51023: MixRGB constant folding not effective with clamp option.

Fix T51855: Cycles emssive objects with NaN transform break lighting.

Fix T51889: broken UI after File > New without a userpref.blend.

BLO_update_defaults_userpref_blend() was being run when no user preferences
were loaded from memory.

Fix T51734: batch-generate previews crashes on some materials.

Fix T51759: fluid simulation particles not remoevd when colliding with outflow objects.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D2719

Fix T51951: cell noise texture precision issue at unit vertex coordinates.

Solution is to bias the coordinates a little, same as Cycles checker texture.

Fix T51746: normal map tangents not working correctly when there are no UV maps.

Patch by Alexander, with some refactoring by Brecht.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D2709

Fix T51772: double undo entry for color picker editing.

Fix T51956: color noise with principled sss, radius 0 and branched path.

Fix T51957: principled BSDF mismatches in GLSL viewport.

Fix principled BSDF incorrectly missing subsurface component with base color black.

Fix T51947: failure setting sequence.use_proxy/crop/translation to False

Fix ED_OT_undo_redo operator

Fix T50230: Rigid Body simulation shouldn't step when time is beyond cached area

This makes the last time (`ltime`) stored in the rigid body world (`rbw`)
only be updated once a simulation step actually occurs, this prevents
another simulation step from being solved unless the current time is
exactly one frame after the last cached frame. Thus this prevents the
formation of gaps in the cache, such as seen in T50230.

Reviewers: mont29, sergey, angavrilov

Tags: #physics

Maniphest Tasks: T50230

Differential Revision: https://developer.blender.org/D2458

Better fix for isfinite problem - works in older gcc's too.

Previous fix relying on __cplusplus value didn't fix for older
gcc's on linux, but this fix does.

Fix compile error after recent 9c2bbfb6 commit.

Older C++ compilers use finite instead of isfinite.

Cycles: Use float constants instead of double

Cycles: Disable baking in mega kernel when not in use to improve build times

Fix T50887: Holes in fluid mesh on Windows

D2556 by @chrisr

Fix unpredictable trackball rotation

D2729 by @IgorNull

Currently, trackball rotation sequentially applies rotation across x axis and y axis,
which produces a strange/unusable result on diagonal pointer motion.

This change fixes the problem by using a single axis which is orthogonal
and proportional to mouse delta - matching view-port trackball.

Workaround T51934: Array's > SHRT_MAX aren't supported

Fix T51890: Turning off viewport display for array modifier is blocking dupliface in final render

DupliVert/Face were not using render DM at render time.

Fix T51926: Selecting pose icon under expanded group in outliner causes crash.

Cannot switch uninstantiated armature to Pose mode...

Fix T51909: Cycles: Uninitialized closure normals for the Hair BSDF

As the title says, the normal wasn't set for the Hair BSDF because it wasn't
needed before. However, the denoiser uses it to store the feature passes, so
it needs to be set now.

Fix node UI not using translation context correctly.

Now that some node types may have custom context, we need to handle that
in the (convoluted :| ) UI code of nodes as well.

Reported in T43295 by Gabriel Gazzán (@gab3d), thanks.

Fix own really stupid mistake/typo in previous commit...

Fix T51900: Crash after pressing "F" multiple times.

`BMO_iter_as_array()` may fill less items than requested in given array,
so we have to update number of items to work on from its returned value,
otherwise code might try to use uninitialized memory.

Fix T51913: Context tab for textures issue

The original code was doing a sanity check to see if existing index was
out of range. However the comparison was wrong.

So if the previous ct->user (active index of texture node) was larger
than then number of available texture nodes + 1 in the other material,
we would never re-set the index to 0.

Bug introduced on c31f74de6bb7.

There was an early attempt of fixing this (2b2ac5d3cc) but it was just working
by pure, luck. And failing in cases like the one from this bug report.

Cleanup: remove unused function

CMake: Only set CMAKE_BUILD_TYPE_INIT when not set

Convenience makefile now uses CMAKE_BUILD_TYPE_INIT,
this means you can change the build type of an existing build
and it won't be overwritten when running `make`.

Useful if you want to add debug info to a release build for profiling.

Add warning that 'purge all' action in Outliner does save to current .blend file.

This is a very important, potentially deadly side-effect of this
operator. If something goes wrong, it can save a broken .blend file.

Ideally we could get rid of that operation anyway, once ID management if
fully renewed, but for now would rather keep it around.

Related to T51902.

Fix T51902: Severe problem with relocating linked libraries when using proxies.

Fix is a bit ugly, but cannot think of another solution for now, at
least this **should** not break anything else.

And now I go find myself a very remote, high and lonely mountain, climb
to its top, roar "I hate proxies!" a few times, and relax hearing the echos...

Enable optional jack in install_deps.sh

Based on D2578, now you can install JACK audio server and use it in
Blender build without having to specify the `--with-all` option (that
one still enables also JACK of course).

Reviewers: mont29

Maniphest Tasks: T51033

Differential Revision: https://developer.blender.org/D2578

Building w/o Python works again

Fix freestyle lineset panels being animatable

UI: Fix Capitalization Inconsistency

Docs: correct doc-strings

Collada: Exporter now only exports either 'Materials' or 'UV Textures as Materials'. This makes the user interface more streight

fix: overlooked the RET_ZERO appendix

fix: T51622 The exporter now exports meshes as <Triangles> when all contained polygons are tris

fix: Collada fprintf needs std::string be converted to char *

fix: collada: removed unnecessary extra qualification

fix: adjusted collada declaration after changes in collada module.
@campbell Barton: Why is this declaration needed at all in stubs.c?
Further up the file collada.h is imported and that already decalres
the function and results in a duplicate declaration.

Collada: Fix: Geometry exporter did not create all polylist when meshes are only partially textured

collada: export UV Textures as materials. Note: the reimport of the exported collada files will have materials assigned instead of UV Face Textures! This is expected behavior

collada: change image export: export either UV Textures or Materials. This
avoids wrong texture data when multiple objects are exported. Note: This
commit might possiblyt not work fully. The full feature is added with the
next commit)

Collada: improved Error handling: Avoid shutdown of Blender when the Collada importer finds Syntax errors in import data

Collada: added new functions for improved material exporter (not used yet)

Fix T50856: crash when minimizing window on AMD / Windows.

Fix T51792: crash calling bpy.ops.screen.animation_cancel while scrubbing.

Grease Pencil: Fix hardcoded DKEY for continous drawing

Fix T51845

Fix T51867: Insert Keyframe I - L / I - R / I - S key combos are broken.

Reorder keyingsets registration order, since it also afects order of I
menu options, better show most used ones first, pure alphabetical order
is not great here... Will likely break some muscle memory though. :|

Based on D2720 by Carlo Andreacchio (@candreacchio), thanks.

Fix T51863: CompositorNodeSwitchView have the wrong rna API

Although the original report was about the docs, the real issue was in
the API.

My original commit started from a copy-paste from the Switch
Node. However I don't use custom1 for thew Switch View node.

The docs is slightly incomplete since it would be nice to mention the
views here. Or maybe even expose them via Python. But honestly they are
generated depending on the scene multi-view settings.

Cycles: Fix excessive sampling weight of glossy Principled BSDF components

If there was any specularity in the Principled BSDF, it would get a sampling
weight of one regardless of its actual impact.

This commit makes Cycles estimate the contribution of the component and adjust
the weighting accordingly, which greatly improves the noise characteristics of
the Principled BSDF in many cases.

Note that this commit might slightly change the brightness of areas when using
MultiGGX and high roughnesses, but the new brightness is more accurate and
closer to the result of Branched Path Tracing. See T51836 for details.

Differential Revision: https://developer.blender.org/D2677

Fix T51836: Cycles: Fix incorrect PDF approximations of the MultiGGX closures

The PDF of the MultiGGX sampling is approximated by the singlescattering GGX
term as well as a scaled diffuse term that makes up for the energy in the
multiscattering component that's missed by GGX.

However, there were two problems with the glossy terms: The diffuse term missed
a normalization factor, and the singlescattering term was not properly scaled
down based on the albedo estimate.

The glass term was completely wrong and has been rewritten. It uses the fresnel
factor to weight reflection vs. refraction and uses the glossy MultiGGX model
for reflection.
For refraction, the correct singlescattering term is now used, and a new
albedo approximation is used that was derived by evaluating GGX albedo for
roughnesses from 0 to 1 and IORs from 1 to 3 and fitting numerical
approximations to it. The resulting model has a mean relative error of 9e-5,
but could probably be simplified without losing noticable accuracy in the
final render.

The improved PDFs help with glossy highlights (due to better light sampling vs.
closure sampling MIS) and fix the situation described in T51836 where mixing
MultiGGX with other closures (as it happens in e.g. the Principled
BSDF) causes incorrect darkening.

Fix T51862: principled shader GLSL artifacts in ortho mode.

Fix T51849: change Cycles clearcoat gloss to roughness.

This is compatible with UE4 and more consistent with specular and transmission
roughness, even if it deviates from the original Disney BRDF.

Fix typo in builtbot config

Fix T51856: `BKE_mesh_new_from_object()` would often generate default 'Mesh' named datablock.

This is annoying especially for exporters who do use mesh name, since it
broke any relation with actual Mesh naming in original Blend file.

Unfortunately, we cannot avoid the extra .xxx digits. ;)

Fix: Icon alignment for scaled pie buttons with no text

Fix (unreported) missing Image usercount increase when copying UVProject modifier.

Alembic tests: make failures a bit easier to diagnose.

Fix T51762: Unit test script_alembic_import is failing.

Implemented workaround for use with the legacy depsgraph.

Fixed bl_load_py_modules / script_load_modules unit test

It tried to assert that
addons/io_blend_utils/blender_bam-unpacked.whl/__init__.py was loaded when
the io_blend_utils module was imported. However, this happens only on
demand, and not directly when importing the add-on.

Use for/else instead of setting 'ok' variable.

This is more efficient, and this use case is exactly what the else clause
is for.

Removed trailing spaces

Fix (unreported) bad copying code of Surface Deform modifier.

Fix compiler warnings from own recent rB0d5c7e5e36b9.

Fix (unreported) bad copying of Ocean modifier.

Was needlessly complicated code, forgot to copy a value (foam_fade), and
was utterly leaking memory!

Fix (unreported) bad copying code in Mesh Deform modifier.

Fix (unreported) Dynamic Paint modifier not increasing ID usercount in copy function.

*Sigh* One more example of why we should keep ID management handling in
as few places as possible! It's impossible to keep more than a few
places in sync regarding which ID pointer is refcounted etc.

Fix (unreported) memory leak in Fluid modifier copying.

Also generally simplify/sanitize this copy code.

Cleanup: doxygen comments

Also remove duplicate & mismatching comments from grease-pencil header.
Keep comments close to implementation to avoid getting out of sync.

Fix T51774: Children particles hair interpolation not correct with textures or dp.

Children where always getting at least one segment of fixed length...

Now fully hidden ones (zero length) get no segment at all.

Note that even very short ones keep getting one 'unit' length segment - would
rather avoid changing that at this point, given how complex children
particles 'length' can get with all kind of modifiers... Think we can
live with that for now anyway.

Fix T51840: UI redraw in node editor header missing on pointcache bake

Missing a notifier handler in the node editor

Fix unreported: Copy-pasting nodes crashes when they have an undefined type

Docs: Fix file:line links in generated API docs