From 0372ff2ba735f3559953753ec7f17fb8bde4ec57 Mon Sep 17 00:00:00 2001 From: Julien Duroure Date: Mon, 4 May 2020 18:34:59 +0200 Subject: glTF: code cleanup - remove no more needed import --- io_scene_gltf2/__init__.py | 2 +- io_scene_gltf2/blender/com/gltf2_blender_conversion.py | 3 +-- io_scene_gltf2/blender/com/gltf2_blender_extras.py | 2 +- io_scene_gltf2/blender/com/gltf2_blender_math.py | 3 ++- io_scene_gltf2/blender/exp/gltf2_blender_extract.py | 1 - io_scene_gltf2/blender/exp/gltf2_blender_gather_accessors.py | 4 ---- .../blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py | 1 - io_scene_gltf2/blender/exp/gltf2_blender_gather_cameras.py | 1 - io_scene_gltf2/blender/exp/gltf2_blender_gather_drivers.py | 2 -- io_scene_gltf2/blender/exp/gltf2_blender_gather_image.py | 2 -- io_scene_gltf2/blender/exp/gltf2_blender_gather_joints.py | 1 - .../exp/gltf2_blender_gather_material_normal_texture_info_class.py | 1 - .../exp/gltf2_blender_gather_materials_pbr_metallic_roughness.py | 1 - .../imp/gltf2_blender_KHR_materials_pbrSpecularGlossiness.py | 1 - io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_unlit.py | 1 - io_scene_gltf2/blender/imp/gltf2_blender_animation.py | 2 -- io_scene_gltf2/blender/imp/gltf2_blender_animation_weight.py | 2 +- io_scene_gltf2/blender/imp/gltf2_blender_gltf.py | 2 +- io_scene_gltf2/blender/imp/gltf2_blender_mesh.py | 2 -- io_scene_gltf2/blender/imp/gltf2_blender_primitive.py | 1 - io_scene_gltf2/blender/imp/gltf2_blender_scene.py | 3 +-- io_scene_gltf2/blender/imp/gltf2_blender_texture.py | 2 +- io_scene_gltf2/io/exp/gltf2_io_export.py | 6 ++---- io_scene_gltf2/io/imp/gltf2_io_binary.py | 1 - io_scene_gltf2/io/imp/gltf2_io_gltf.py | 2 +- 25 files changed, 12 insertions(+), 37 deletions(-) diff --git a/io_scene_gltf2/__init__.py b/io_scene_gltf2/__init__.py index 2e06ae92..927f0153 100755 --- a/io_scene_gltf2/__init__.py +++ b/io_scene_gltf2/__init__.py @@ -15,7 +15,7 @@ bl_info = { 'name': 'glTF 2.0 format', 'author': 'Julien Duroure, Norbert Nopper, Urs Hanselmann, Moritz Becher, Benjamin Schmithüsen, Jim Eckerlein, and many external contributors', - "version": (1, 2, 73), + "version": (1, 2, 74), 'blender': (2, 83, 9), 'location': 'File > Import-Export', 'description': 'Import-Export as glTF 2.0', diff --git a/io_scene_gltf2/blender/com/gltf2_blender_conversion.py b/io_scene_gltf2/blender/com/gltf2_blender_conversion.py index 574e26e5..c8d2dc8f 100755 --- a/io_scene_gltf2/blender/com/gltf2_blender_conversion.py +++ b/io_scene_gltf2/blender/com/gltf2_blender_conversion.py @@ -12,8 +12,7 @@ # See the License for the specific language governing permissions and # limitations under the License. -from mathutils import Matrix, Quaternion -from math import sqrt, sin, cos +from math import sin, cos def texture_transform_blender_to_gltf(mapping_transform): """ diff --git a/io_scene_gltf2/blender/com/gltf2_blender_extras.py b/io_scene_gltf2/blender/com/gltf2_blender_extras.py index 6cf220e6..6474e3f0 100644 --- a/io_scene_gltf2/blender/com/gltf2_blender_extras.py +++ b/io_scene_gltf2/blender/com/gltf2_blender_extras.py @@ -89,5 +89,5 @@ def set_extras(blender_element, extras, exclude=[]): try: blender_element[custom_property] = value - except TypeError as e: + except TypeError: print('Error setting property %s to value of type %s' % (custom_property, type(value))) diff --git a/io_scene_gltf2/blender/com/gltf2_blender_math.py b/io_scene_gltf2/blender/com/gltf2_blender_math.py index b3f13f65..72eb124a 100755 --- a/io_scene_gltf2/blender/com/gltf2_blender_math.py +++ b/io_scene_gltf2/blender/com/gltf2_blender_math.py @@ -12,7 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -import bpy import typing import math from mathutils import Matrix, Vector, Quaternion, Euler @@ -171,6 +170,7 @@ def round_if_near(value: float, target: float) -> float: """If value is very close to target, round to target.""" return value if abs(value - target) > 2.0e-6 else target + def scale_rot_swap_matrix(rot): """Returns a matrix m st. Scale[s] Rot[rot] = Rot[rot] Scale[m s]. If rot.to_matrix() is a signed permutation matrix, works for any s. @@ -183,6 +183,7 @@ def scale_rot_swap_matrix(rot): m[i][j] = abs(m[i][j]) # discard sign return m + def nearby_signed_perm_matrix(rot): """Returns a signed permutation matrix close to rot.to_matrix(). (A signed permutation matrix is like a permutation matrix, except diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_extract.py b/io_scene_gltf2/blender/exp/gltf2_blender_extract.py index e546b063..35f31c50 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_extract.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_extract.py @@ -24,7 +24,6 @@ from . import gltf2_blender_export_keys from ...io.com.gltf2_io_debug import print_console from ...io.com.gltf2_io_color_management import color_srgb_to_scene_linear from io_scene_gltf2.blender.exp import gltf2_blender_gather_skins -import bpy # # Globals diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_accessors.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_accessors.py index 2fe992bc..58cbe158 100644 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_accessors.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_accessors.py @@ -13,14 +13,10 @@ # limitations under the License. -import typing - -import bpy from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import cached from io_scene_gltf2.io.com import gltf2_io from io_scene_gltf2.io.com import gltf2_io_constants from io_scene_gltf2.io.exp import gltf2_io_binary_data -from . import gltf2_blender_export_keys @cached diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py index 6fa5a11d..822aa6a1 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_animation_sampler_keyframes.py @@ -19,7 +19,6 @@ import typing from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import cached, bonecache from io_scene_gltf2.blender.com import gltf2_blender_math from io_scene_gltf2.blender.exp import gltf2_blender_get -from io_scene_gltf2.blender.exp import gltf2_blender_extract from io_scene_gltf2.blender.exp.gltf2_blender_gather_drivers import get_sk_drivers, get_sk_driver_values from . import gltf2_blender_export_keys from io_scene_gltf2.io.com import gltf2_io_debug diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_cameras.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_cameras.py index c9926392..585f0be3 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_cameras.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_cameras.py @@ -12,7 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -from . import gltf2_blender_export_keys from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import cached from ..com.gltf2_blender_extras import generate_extras from io_scene_gltf2.io.com import gltf2_io diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_drivers.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_drivers.py index b176ca45..c0bc2cc6 100644 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_drivers.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_drivers.py @@ -13,8 +13,6 @@ # limitations under the License. -from io_scene_gltf2.blender.exp import gltf2_blender_gather_nodes -from io_scene_gltf2.blender.com import gltf2_blender_conversion from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import skdriverdiscovercache, skdrivervalues from io_scene_gltf2.blender.com.gltf2_blender_data_path import get_target_object_path diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_image.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_image.py index bd801430..69ebcb66 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_image.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_image.py @@ -11,12 +11,10 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. -import re import bpy import typing import os -import numpy as np from . import gltf2_blender_export_keys from io_scene_gltf2.io.com import gltf2_io diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_joints.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_joints.py index f7eed77f..d5ecc8da 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_joints.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_joints.py @@ -17,7 +17,6 @@ import mathutils from . import gltf2_blender_export_keys from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import cached from io_scene_gltf2.io.com import gltf2_io -from io_scene_gltf2.io.com import gltf2_io_debug from io_scene_gltf2.blender.exp import gltf2_blender_extract from io_scene_gltf2.blender.com import gltf2_blender_math from io_scene_gltf2.blender.exp import gltf2_blender_gather_skins diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_material_normal_texture_info_class.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_material_normal_texture_info_class.py index b554717b..e0b9be21 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_material_normal_texture_info_class.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_material_normal_texture_info_class.py @@ -18,7 +18,6 @@ from io_scene_gltf2.blender.exp.gltf2_blender_gather_cache import cached from io_scene_gltf2.io.com import gltf2_io from io_scene_gltf2.blender.exp import gltf2_blender_gather_texture from io_scene_gltf2.blender.exp import gltf2_blender_search_node_tree -from io_scene_gltf2.blender.exp import gltf2_blender_export_keys from io_scene_gltf2.blender.exp import gltf2_blender_get from io_scene_gltf2.io.com.gltf2_io_extensions import Extension from io_scene_gltf2.io.exp.gltf2_io_user_extensions import export_user_extensions diff --git a/io_scene_gltf2/blender/exp/gltf2_blender_gather_materials_pbr_metallic_roughness.py b/io_scene_gltf2/blender/exp/gltf2_blender_gather_materials_pbr_metallic_roughness.py index e774bfef..ebef6597 100755 --- a/io_scene_gltf2/blender/exp/gltf2_blender_gather_materials_pbr_metallic_roughness.py +++ b/io_scene_gltf2/blender/exp/gltf2_blender_gather_materials_pbr_metallic_roughness.py @@ -13,7 +13,6 @@ # limitations under the License. import bpy -from mathutils import Color from io_scene_gltf2.io.com import gltf2_io from io_scene_gltf2.blender.exp import gltf2_blender_gather_texture_info, gltf2_blender_search_node_tree diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_pbrSpecularGlossiness.py b/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_pbrSpecularGlossiness.py index 7669d986..ce5e1aed 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_pbrSpecularGlossiness.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_pbrSpecularGlossiness.py @@ -12,7 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -import bpy from ...io.com.gltf2_io import TextureInfo from .gltf2_blender_pbrMetallicRoughness import \ base_color, emission, normal, occlusion, make_output_nodes, make_settings_node diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_unlit.py b/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_unlit.py index f1a8fefe..cd0a8568 100644 --- a/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_unlit.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_KHR_materials_unlit.py @@ -12,7 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -import bpy from .gltf2_blender_pbrMetallicRoughness import base_color, make_output_nodes diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_animation.py b/io_scene_gltf2/blender/imp/gltf2_blender_animation.py index f01e5ee9..93581542 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_animation.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_animation.py @@ -12,8 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -import bpy - from .gltf2_blender_animation_node import BlenderNodeAnim from .gltf2_blender_animation_weight import BlenderWeightAnim from .gltf2_blender_animation_utils import simulate_stash, restore_animation_on_object diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_animation_weight.py b/io_scene_gltf2/blender/imp/gltf2_blender_animation_weight.py index 5ce4532e..1bf67213 100644 --- a/io_scene_gltf2/blender/imp/gltf2_blender_animation_weight.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_animation_weight.py @@ -16,7 +16,7 @@ import json import bpy from ...io.imp.gltf2_io_binary import BinaryData -from .gltf2_blender_animation_utils import simulate_stash, make_fcurve +from .gltf2_blender_animation_utils import make_fcurve class BlenderWeightAnim(): diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_gltf.py b/io_scene_gltf2/blender/imp/gltf2_blender_gltf.py index e00e2449..efa7f003 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_gltf.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_gltf.py @@ -121,7 +121,7 @@ class BlenderGlTF(): # Dispatch animation if gltf.data.animations: - for node_idx, node in enumerate(gltf.data.nodes): + for node in gltf.data.nodes: node.animations = {} track_names = set() diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_mesh.py b/io_scene_gltf2/blender/imp/gltf2_blender_mesh.py index adf05281..7914a41b 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_mesh.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_mesh.py @@ -14,12 +14,10 @@ import bpy import bmesh -from mathutils import Vector from ..com.gltf2_blender_extras import set_extras from .gltf2_blender_material import BlenderMaterial from .gltf2_blender_primitive import BlenderPrimitive -from ...io.imp.gltf2_io_binary import BinaryData class BlenderMesh(): diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_primitive.py b/io_scene_gltf2/blender/imp/gltf2_blender_primitive.py index 95688055..d544778c 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_primitive.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_primitive.py @@ -15,7 +15,6 @@ import bpy from mathutils import Vector, Matrix -from .gltf2_blender_material import BlenderMaterial from ...io.imp.gltf2_io_binary import BinaryData from ...io.com.gltf2_io_color_management import color_linear_to_srgb from ...io.com import gltf2_io_debug diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_scene.py b/io_scene_gltf2/blender/imp/gltf2_blender_scene.py index 1bc3fe0b..6119eb95 100755 --- a/io_scene_gltf2/blender/imp/gltf2_blender_scene.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_scene.py @@ -13,8 +13,7 @@ # limitations under the License. import bpy -from math import sqrt -from mathutils import Quaternion + from .gltf2_blender_node import BlenderNode from .gltf2_blender_animation import BlenderAnimation from .gltf2_blender_vnode import VNode, compute_vnodes diff --git a/io_scene_gltf2/blender/imp/gltf2_blender_texture.py b/io_scene_gltf2/blender/imp/gltf2_blender_texture.py index 58d101d0..9dc012a2 100644 --- a/io_scene_gltf2/blender/imp/gltf2_blender_texture.py +++ b/io_scene_gltf2/blender/imp/gltf2_blender_texture.py @@ -13,10 +13,10 @@ # limitations under the License. import bpy + from .gltf2_blender_image import BlenderImage from ..com.gltf2_blender_conversion import texture_transform_gltf_to_blender from io_scene_gltf2.io.com.gltf2_io import Sampler -from io_scene_gltf2.io.com.gltf2_io_debug import print_console from io_scene_gltf2.io.com.gltf2_io_constants import TextureFilter, TextureWrap def texture( diff --git a/io_scene_gltf2/io/exp/gltf2_io_export.py b/io_scene_gltf2/io/exp/gltf2_io_export.py index 07cf8820..54181206 100755 --- a/io_scene_gltf2/io/exp/gltf2_io_export.py +++ b/io_scene_gltf2/io/exp/gltf2_io_export.py @@ -103,16 +103,14 @@ def save_gltf(gltf, export_settings, encoder, glb_buffer): file.write(struct.pack("I", length_gltf)) file.write('JSON'.encode()) file.write(gltf_data) - for i in range(0, spaces_gltf): - file.write(' '.encode()) + file.write(b' ' * spaces_gltf) # Chunk 1 (BIN) if length_bin > 0: file.write(struct.pack("I", length_bin)) file.write('BIN\0'.encode()) file.write(binary) - for i in range(0, zeros_bin): - file.write('\0'.encode()) + file.write(b'\0' * zeros_bin) file.close() diff --git a/io_scene_gltf2/io/imp/gltf2_io_binary.py b/io_scene_gltf2/io/imp/gltf2_io_binary.py index 9057c17c..7cfcbc40 100755 --- a/io_scene_gltf2/io/imp/gltf2_io_binary.py +++ b/io_scene_gltf2/io/imp/gltf2_io_binary.py @@ -13,7 +13,6 @@ # limitations under the License. import struct -import base64 from ..com.gltf2_io import Accessor diff --git a/io_scene_gltf2/io/imp/gltf2_io_gltf.py b/io_scene_gltf2/io/imp/gltf2_io_gltf.py index 88ba5356..640e552a 100755 --- a/io_scene_gltf2/io/imp/gltf2_io_gltf.py +++ b/io_scene_gltf2/io/imp/gltf2_io_gltf.py @@ -18,7 +18,7 @@ import logging import json import struct import base64 -from os.path import dirname, join, isfile, basename +from os.path import dirname, join, isfile from urllib.parse import unquote -- cgit v1.2.3