Merge pull request #1445 from Matter-and-Form/gltf2-alphaMode-fix

Fix glTF2 alphaMode storage and reading

Merge pull request #1447 from Matter-and-Form/gltf1-color-import-fix

[gltf1] Don’t ignore rgba(1,1,1,1) color properties

Merge pull request #1446 from Matter-and-Form/feature/gltf2-primitives

glTF2 primitives fixes

Merge branch 'master' of https://github.com/assimp/assimp

FBX: add missing inversion of postrotation matrix for fbx.

Don’t ignore rgba(1,1,1,1) color properties

Do not ignore rgba(1,1,1,1) material properties when importing glTF1. While a white diffuse color may be the default value for a default assimp material, `1,1,1,1` is a very explicit color value for ambient/specular/emissive color properties.

Closes #1434

Formatting

Fix alphaMode storage and reading

alphaMode is now converted from a std::string to an aiString and back to std::string, since aiString is easier to store and retrieve from aiMaterial properties than std::string

Fixes issues of alphaMode being written out as `\fOPAQUE\0\0\0\0\0\0…`

SImplify mesh merging code

My assumption that primitives of different types (modes) can’t be in the same mesh was incorrect.

tweaks to primitive merging logic; comments + formatting

Formatting

Merge multiple meshes in a node into one mesh with many primtives; write out only one mesh per node

To do:
- clean up MergeMeshes
- see if there’s a way to do this earlier in the flow

Revert "store node mesh vs. meshes"

This reverts commit a0d97505e5a53a9aa3a734d6bd3ad790234e6e09.

Merge pull request #1442 from jcowles/master

Fix glTF 2.0 multi-primitive support

Fix glTF 2.0 multi-primitive support

Previously, only one primitive was supported, in fact memory was corrupted
when more than one primitive was found per glTF mesh.

In this change, each primitive is unrolled as a new Assimp Mesh, resulting
in multiple Assimp meshes per node when multiple primitives exist per
glTF mesh. This is required in the general case, since glTF primitives can
have different material bindings and primitive modes.

Merge pull request #1441 from turol/travis

Disable Travis OS X build since it doesn't do anything currently

Travis: Disable OS X build since it doesn't do anything currently

Merge pull request #1440 from turol/travis

Travis cleanup

Merge pull request #1435 from jaredmulconry/issue_1065

Last remaining build warning emitted from MSVC for x64 builds

Merge pull request #1437 from rmitton/sib-version

Added support for SIB models from Silo 2.5

travis: Enable ccache

travis: Only build with xcode 8.3

travis: Correctly minimize build matrix

travis: Move os declarations earlier

Merge pull request #1436 from turol/warnings

Clean up some GCC warnings

Added Silo 2.5 support

Silo 2.5 bumps the version number of SIB files for no apparent reason. Doesn't appear to be any other changes to the file format.

Fixed truncated material names

The previous Unicode library change accidentally trimmed off the last character of SIB material names.

Merge pull request #1 from assimp/master

Merged from master

FBXParser: Silence uninitialized variable warnings

Importer: Whitespace cleanup to fix GCC misleading indentation warning

AssbinExporter: Fix strict aliasing violation

AssbinExporter: Add Write specialization for aiColor3D

DefaultLogger: Whitespace cleanup to fix GCC misleading indentation warning

glTFExporter: Silence uninitialized variable warning

This is a false positive, idx_srcdata_begin is only used if comp_allow
is true and in that case it's also initialized.

Add CMake flag to treat warnings as errors

Addressed last remaining warning under MSVC caused by use of 'deprecated' fopen.

Merge pull request #1433 from vkovalev123/patch-1

Update 3DSLoader.cpp

Update 3DSLoader.cpp

Fixed reading of CHINK_RGBF. If reading performs on x32 platform then reading will execute right, but on x64 it`s wrong because it will read 8 bytes instead 4.

Merge pull request #1431 from assimp/revert-1427-asan

Revert "Asan"

Revert "Asan"

Merge pull request #1423 from Matter-and-Form/feature/gltf2

glTF 2 Importer + glTF 2 Spec Conformity

Merge pull request #1427 from turol/asan

Asan

Update glTF in list of importers and exporters

Fix delete / delete[] mismatch in glTFAsset

Enable AddressSanitizer for Linux clang build

MDP: fix encoding issues.

Merge branch 'master' of https://github.com/assimp/assimp

FBX: fix some minor findings.

closes https://github.com/assimp/assimp/issues/1426: add Defines.h to include folder for install.

Add AddressSanitizer option to CMake

Manually read alphaMode material property

Properly move string passed to JSON writer

Fix Segfault caused by losing pointer to std::string

Keep std::string alive

Revert "Remove simple gltf2 export unit test"

This reverts commit 4b01ecaf109ae1bbd3ae918e30abdc67d7448a5a.

Remove simple gltf2 export unit test

Segfaults on Linux for some reason. No other tests test exporting, so it’s fine

Exclude glTF2 Exporter test when ASSIMP_BUILD_NO_EXPORT

Formatting

Use `forceNumber` argument of `WriteAttrs` to write correct attribute names, instead

Fix mesh primitive’s attributes’ names

Per spec

TEXCOORD -> TEXCOORD_0
COLOR -> COLOR_0
JOINTS -> JOINTS_0
WEIGHTS -> WEIGHTS_0

Remove JOINTMATRIX since it’s not supported (and doesn’t seem to be output, anyway)

TANGENT should be added at a later date

Remove KHR_binary_glTF code

Binary glTF is now part of the glTF2 spec. However, it’s implemented incorrectly, so will be temporarily removed

Remove compresssed file format flag

Add gltf2 basic unit test

more specific token search for Collada Loader

collada can appear in many files, such as glTFs via the “generator” field (in the form of collada2gltf)

Load gltf .bin files from correct directory

Remove premultipliedAlpha from gltf2

Fix parsing of glTF version
Handle version as int in gltf
Fix format specifiers in glTF version parser

Add support for importing both glTF and glTF2 files

Restrict search for OFF header to first 3 bytes

Remove commented out code

Set alphaMode, baseColorFactor opacity when model’s opacity isn’t 1

Set the metallicFactor to 0 if source file doesn’t have metallicFactor

Otherwise, the models have full metallic materials and appear very dim

Fix METALLIC_FACTOR typo

Define gltf material property names as constants

Fix indentation

Implement pbrSpecularGlossiness property as Nullable

Define default material values as static constants

for reuse

Remove need for Has by returning an empty Ref in Get

Use different form of index accessor

Export material names properly

Sampler improvements; Add new LazyDict method

Samplers are now imported into assimp internal format from textures;
Samplers have defaults as per spec;
Sampler enums are strongly typed with UNSET values;
Samplers are exported properly;
Sampler filters are exported as well;
Samplers are re-used across textures on export
Default sampler values are not written

Set default values on Sampler

Fix unused CopyValue

Export extensions

Formatting

Only export byteStride if not 0

store node mesh vs. meshes

glTF nodes can only hold one mesh. this simply assigns to and check’s a Node’s Mesh

reenable animation and skins exports

Currently incorrect, however. May need to be removed

Write specularGlossiness textures on the specularGlossiness object

Make sure `on` flag for specularGlossiness is being persisted

mention pbrSpecularGlossiness support

Remove OPEN3DGC and compression references

Changes to GLTF2 materials

pbrMetallicRoughness and pbrSpecularGlossiness as structs; persist textureinfo properties from start to finish; persist pbrSpecularGlossiness (via extensionsUsed) usage from start to finish

Remove Light, Technique references

Start removing materials common, and adding pbrSpecularGlossiness

Remove redundant function

Use `!ObjectEmpty()` vs. `MemberCount() > 0`

Working read, import, export, and write of gltf2 (pbr) material

check in gltf2 models to test directory
Remove un-needed test models

Start managing and importing gltf2 pbr materials

temporarily disable gltf exporting of animations and skins