Alembic: don't call object.getFullName() multiple times.

Alembic: Renamed parent_map to reader_map

It's a mapping from full path of an Alembic object to an AbcObjectReader*.
The fact that at some point it is used to construct parent-child relations
doesn't matter.

Alembic: use typedefs to make it easier to handle maps/vectors

Alembic: more const-ness

Alembic: fixed importer

The importer was guessing whether an Alembic IXform object was part of a
child object, or should be represented as an Empty in Blender. By reversing
the order in which objects are visited, the children can now claim their
parent as part of the same object (so IPolyMesh claims its parent IXform
as part of the same Blender object). This results in much less guesswork.

I've also removed similar guesswork from the code that sets parent pointers,
by simply searching for the parent in a hierarchical way, instead of trying
to predict (again) which IXforms were turned into empties.

Also, visit_object() now actually visits the object -- previously it only
visited its children, and assumed the object it was called on was already
handled by a previous call.

Alembic: prevent a couple of unnecessary multiplications

Alembic: be more explicit in y-up versus z-up variables.

Alembic: removed a lot of unnecessary & duplicate code from abc_util.cc

create_transform_matrix(float[4][4]) did mostly the same as
create_transform_matrix(Object *, float[4][4]), but more elegant.
However, the former has some inconsistencies with the latter (which
are now merged and made explicit, turned out one was for z-up→y-up
while the other was for y-up→z-up), and was renamed to
copy_m44_axis_swap(...) to convey its purpose more clearly.

Furthermore, "loc" has been renamed to "trans", as matrices don't
store locations but translations; and more variables now have a src_
or dst_ prefix to denote whether they contain a matrix/vector in the
source or destination axis orientation.

Alembic: added note

Alembic: allow copy_{z,y}up_from_{y,z}up() to be called with yup=zup

This allows in-place conversion between z-up and y-up, by passing the
same variable to both arguments.

Alembic: import empties under their own name in Alembic, not their parent's

Before this commit something strange happened, as the m_data_name of
an inherit data-less object was used.

Alembic: Don't compute world matrix when it's not necessary.

Alembic: cleaned up hack in AbcObjectReader::read_matrix()

Alembic: added some clarification

Alembic: added some getter functions to help debug prints etc.

Alembic: do not presume to predict the parent name

AbcExporter::createTransformWriter() tries to predict the parent Xform
name, but if it cannot be found has multiple ways of creating it, possibly
under a different name than originally searched for.

Alembic: Removed AbcTransformWriter::m_parent

It was set, but never read anywhere.

Buildbot: Correct previous release

Seems CMake is not happy about changing compiler from script.

Buildbot: Force build environment to use latest GCC

Cleanup: function naming for manipulator

Rename 'stats_*' to 'protectflag_to_drawflags_*' (was too vague).
Also remove NULL check from gimbal_axis

Cleanup: minor changes to transform-manipulator

Match stats_editbone & stats_pchan behavior to avoid confusion.

Fix alembic build error with MSVC

Depsghraph: Remove unused function

Cycles: Fix race condition in attributes creation during SVM compilation

Cycles: Add utility class to simplify scoped spin locks

Cycles: Guard global write access in SVM compilation code

Locales: Some optimization work

Mainly visible for MSVC debug builds and gives about 2x speedup.

InstallDeps: Raise minimal & official version of OIIO to 1.7.13.

As requested by @sergey.

Alembic: addition of a simple logging class.

The idea is to have a system where we properly log error messages and
let the users know that errors occured redirecting them to the console
for explanations. This is only implemented for the exporter since the
importer already has similar functionalities; however they shall
ultimately be unified in some way.

Reviewers: sybren, dfelinto

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

Cleanup: redundant casts

Fix missing protection of `RNA_pointer_as_string()` against NULL pointers.

Odd that issue was never reached before? Looks like it hit in
datablock_idprops branch though...

Cycles: Tag shaders for update after the threading part is over

This avoids write access happening in non-atomic manner in
Shader::tag_update which modifies the global managers. Even
for 1 byte data types it's quite dangerous.

Depsgraph: Use atomic operation to tag the changed ID

Cycles: Fix uninitialized memory access when comparing curve mapping nodes

The issue is coming from the fact that float3 is actually 16 bytes aligned
data type and the "padding" was not initialized. This caused memcmp() to
access non-initialized memory.

install_deps: removed leftover compile_HDF5 command

It was a leftover from when Alembic with HDF5 was still officially
supported.

Bumped Alembic library version to 1.7.1

This provides us with a clearer API (so I don't have to use const_cast<>
in upcoming code). It also allows layering of different Alembic files,
so you can have a base file and load a separate file containing overrides.

Verbally approved by Dr. Sergey.

Buildbot: Update master config

Alembic: force ALEMBIC_LIB_USES_BOOST=ON when not using C++11

Alembic requires one of ALEMBIC_LIB_USES_BOOST, ALEMBIC_LIB_USES_TR1, or
C++11, and silently defaults to the latter if the former two are OFF.

Before this change, Alembic was only built without C++11 of OpenEXR
was built at the same time. This dependency was both unnecessary and
undocumented.

Task: Remove non-atomic pool suspended flag assignment

This was done some lines above by atomic fetch and and.

Buildbot: Update master config

Buildbot: Some more twqeaks to master config

Buildbot: Update bundled vetrsion of server configuration

Buildbot: SPecial branch trickery for linux slaves

Point all submodules to master branch

This way it should be safe to use `git submodule update --remote`.

Buildbot: Some special tricks for Blender 2.8 slave

Fix blender player

Fix T51115: Bump node is broken when the displacement socket is used

Collada - add flag to limit precision of exported data, mainly to simplify debugging

fix: collada - do proper conversion from int to bool (as with other nearby parameters)

fix: T50412 - collada: Replaced precision local limit function by blender's own implementation

Cleanup: style

Depsgraph: Correction for the previous local view commit

Need to flush layers from components back to ID node.

Fix threading conflicts in multitex_ext_safe()

This function was modifying texture datablock, which makes the call
unsafe for call from multiple threads. Now we pass the argument that
we don't need nodes to the underlying functions.

There will be still race condition in noise texture, but that should
at least be free from crashes. Doesn't mean we shouldn't fix it tho.

Cycles: Solve threading conflict in shader synchronization

Update tag might access links (when checking for attributes) and
the links might be in the middle of rebuild in simplification
logic.

Fix unreported bug in Blender Render: using unnormalized normal in normal map node in the same way as in baking

WM: Previous commit broke common-case loading new file

Handle this in the operator

Keep current app-template when selecting 'New File'

UI cleanup: simplify Icon handling of uiDefAutoButR for PROP_POINTER.

Comes from D113, but really not related to the patch's topic!

Cycles: More fixes after include changes

Fix T51072: The reference of a pyobject may be being overwritten in `bm_mesh_remap_cd_update`

In this case the Pyobject gets lost from pybm, and bm.free() does not invalidate the PyElem.
This will cause the destructor of python to read invalid memory and crash.

The solution is to make a copy of the pyobjects pointers before overwriting.

Correct string formatting (error in recent change)

Cleanup: warnings

Cleanup/fix bad code in IDP_SetIndexArray()

Mainly, using index before checking for its validity...

Redraw parent popup when the child popup is closed

Depsgraph: Fix missing updates when in local view

This area is a subject of reconsideration, so for now used simplest
way possible -- ensure depsgraph's nodes have proper layer flags
when going in and out of local mode.

Cycles: Fix missing kernel re-compilation after recent changes

Reported by Mai in IRC, thanks!

Fix T51051: Incorrect render on 32bit Linux

The issue was apparently caused by -fno-finite-math-only added to kernel.cpp
CFLAGS. For now just removed this flag from the kernel (we don't really want
it there at this point, and we don't have it for SSE/AVX optimized kernels).

But surely more investigation is needed here.

Cycles: Cleanup, rename RegularBVH to BinaryBVH

Makes it more explicit what the structure is from it's name.

Mirror Modifier: Add offsets for mirrored UVs

The mirror modifier now has two fields that specify a -1 to 1 offset for
the U and V axes when mirroring their coordinates.

D1844 by @circuitfox

Cycles: Fix compilation error of app after the include directories change

Cycles: Attempt to work around compilation errors of CUDA on sm_2x

Cycles: Cleanup, indentation

Cycles: Remove toolkit-specific workaround from kernel

Cycles: Only use CUDA 8.0 as officially supported one

This deprecates CUDA 7.5.

Cycles: Cleanup, don't use m_ prefix for public properties

Cycles: Fix compilation error with visibility flag disabled

Cycles: Make all #include statements relative to cycles source directory

The idea is to make include statements more explicit and obvious where the
file is coming from, additionally reducing chance of wrong header being
picked up.

For example, it was not obvious whether bvh.h was refferring to builder
or traversal, whenter node.h is a generic graph node or a shader node
and cases like that.

Surely this might look obvious for the active developers, but after some
time of not touching the code it becomes less obvious where file is coming
from.

This was briefly mentioned in T50824 and seems @brecht is fine with such
explicitness, but need to agree with all active developers before committing
this.

Please note that this patch is lacking changes related on GPU/OpenCL
support. This will be solved if/when we all agree this is a good idea to move
forward.

Reviewers: brecht, lukasstockner97, maiself, nirved, dingto, juicyfruit, swerner

Reviewed By: lukasstockner97, maiself, nirved, dingto

Subscribers: brecht

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

Cycles: Attempt to workaround compilation error on new CUDA toolkit and sm_2x

Remove non-bmesh case from test

Fix skin mark operator

Accessed custom-data layer offset before creating.

Fix crash closing window in background mode

Fix missing NULL check in gpencil poll

Also de-duplicate poll functions

Use 'empty' option for clearing factory settings

Option to load startup file with empty-data

Useful for batch conversion and tests.

Buildbot: Revert previous change, older toolkit has same exact behavior

Buildbot: Use older NVCC on 32bit linux

Newer toolkit has some weird issue with cross0-compiling 32bit kernels
from 64bit environment.

Buildbot: Remove global hardcoded NVCC path

This was initially needed for heterogeneous setup of two toolkits which
we no longer need.

Cycles: Bring back preview AA samples when using BPT

This was removed in 93426cb. Please be more accurate when
changing interface.

PyAPI: minor path init simplification

Fix 'bl_app_override' wrapping multiple times.

Calling `SomeClass.draw(self, context)` instead of `self.draw()`
would try to wrap the argument `self` multiple times, causing an error.

Fix memory leak re-registering operators

Re-registering an operator used by the keymap would lead memory.
Reload scripts for eg leaked over ~1600 blocks.

PyAPI: check modules are registered before unregister

Needed since templates may unregister classes.

Also replace old modules on reloading.

Fix T51068: Place props in their own row

This allows the props to extend into the blank space that is to the right.

Cycles: Switch to reformulated Pluecker ray/triangle intersection

The intention of this commit it to address issues mentioned in the
reports T43865,T50164 and T50452.

The code is based on Embree code with some extra vectorization
to speed up single ray to single triangle intersection.

Unfortunately, such a fix is not coming for free. There is some
slowdown for AVX2 processors, mainly due to different vectorization
code, which caused different number of instructions to be executed
and different instructions-per-cycle counters. But on another hand
this commit makes pre-AVX2 platforms such as AVX and SSE4.1 a bit
faster. The prerformance goes as following:

              2.78c AVX2   2.78c AVX   Patch AVX2         Patch AVX
BMW            05:21.09     06:05.34    05:32.97 (+3.5%)   05:34.97 (-8.5%)
Classroom      16:55.36     18:24.51    17:10.41 (+1.4%)   17:15.87 (-6.3%)
Fishy Cat      08:08.49     08:36.26    08:09.19 (+0.2%)   08:12.25 (-4.7%
Koro           11:22.54     11:45.24    11:13.25 (-1.5%)   11:43.81 (-0.3%)
Barcelone      14:18.32     16:09.46    14:15.20 (-0.4%)   14:25.15 (-10.8%)

On GPU the performance is about 1.5-2% slower in my tests on GTX1080
but afraid we can't do much as a part of this chaneg here and
consider it a price to pay for more proper intersection check.

Made in collaboration with Maxym Dmytrychenko, big thanks to him!

Reviewers: brecht, juicyfruit, lukasstockner97, dingto

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

Forgot those IDP_LibLinkProperty call on node sockets IDProps in previous commit...

Fix: Icon offset for pie buttons

Cycles: Pass m128 vectors by const reference

Fix lib_link_cachefile.

That one was:
* Resetting non-ID pointers (lib_link_xxx funcs should only affect ID
  pointers, everything else shall be done in direct_link_xxx func).
* Even worse, always calling lib_link_animdata, even when
  LIB_TAG_NEED_LINK tag was unset...

Bring back `lib_link_mesh()` in 'order' with other libdata liblink functions.

We do not need any special handling anymore for usercount of images used
by faces/polygons (tpage stuff), since we have the 'real_user' handling,
which will gracefully cope with all possible situations.

So better not keep that ugly confusing useless special case.

readfile.c: Cleanup lib_link code a bit.

Mainly:
* Add missing `IDP_LibLinkProperty()` calls for many ID types
  (harmless currently, but better be consistent here!).
* Bring lib_link_xxx functions more in line with each other.
* Replace some long if/else by switch.