glTF 2.0 importer / exporter #57763

New Issue

Julien Duroure · 2018-11-10T09:40:27+01:00

Julien Duroure commented

2018-11-10 09:40:27 +01:00

Importer / Exporter of gltf2 files.io_scene_gltf2.zip
This addon is developed by KhronosGroup.

Here is current development git & documentation:
https://github.com/KhronosGroup/glTF-Blender-IO

Importer / Exporter of gltf2 files.[io_scene_gltf2.zip](https://archive.blender.org/developer/F5473163/io_scene_gltf2.zip) This addon is developed by KhronosGroup. Here is current development git & documentation: https://github.com/KhronosGroup/glTF-Blender-IO

Bastien Montagne was assigned by Julien Duroure

2018-11-10 09:40:27 +01:00

Julien Duroure commented

2018-11-10 09:40:27 +01:00

Added subscriber: @JulienDuroure

Julien Duroure commented

2018-11-10 10:57:16 +01:00

For long term development and maintenance, we will have daily changes and fixes in the github repository. If we have a major and/or critical update, we also want to provide and integrate it into future versions of Blender.

Bastien Montagne commented

2018-11-10 12:16:43 +01:00

Hi Julien, please create a differential instead (https://developer.blender.org/differential/diff/create/), it’s much easier to review. (I know the 'submit new addon' creates a new task… :/ ).

Only did a very quick overview, one thing that we should not accept I think is the double 2.7x/2.8x compatibility inside a same code. I understand the logic behind that for an external add-on, but it adds some overhead and clutter to the code, and the ones shipped with Blender should never care about that (since they are only expected to be working with the version they are released with).

Hi Julien, please create a differential instead (https://developer.blender.org/differential/diff/create/), it’s much easier to review. (I know the 'submit new addon' creates a new task… :/ ). Only did a very quick overview, one thing that we should not accept I think is the double 2.7x/2.8x compatibility inside a same code. I understand the logic behind that for an external add-on, but it adds some overhead and clutter to the code, and the ones shipped with Blender should never care about that (since they are only expected to be working with the version they are released with).

Julien Duroure commented

2018-11-10 12:27:48 +01:00

Hello Bastien,

OK, I will create a diff.
Note that zip I submitted has no double compatibility (you probably read files from github that have this double stuff).

Hello Bastien, OK, I will create a diff. Note that zip I submitted has no double compatibility (you probably read files from github that have this double stuff).

Julien Duroure commented

2018-11-10 12:42:20 +01:00

@mont29 Here is the diff : https://developer.blender.org/differential/diff/12493/

Philipp Oeser commented

2018-11-12 09:33:28 +01:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2018-11-12 09:33:28 +01:00

Changed status from 'Open' to: 'Archived'

Philipp Oeser closed this issue

2018-11-12 09:33:28 +01:00

Philipp Oeser commented

2018-11-12 09:33:28 +01:00

Hi @JulienDuroure : thnx for submitting this (very welcome addition).
Since this is now in Differential and in review, I think we can close this task (to only have discussion in one place)?

Hi @JulienDuroure : thnx for submitting this (very welcome addition). Since this is now in Differential and in review, I think we can close this task (to only have discussion in one place)?

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

glTF 2.0 importer / exporter #57763