saturday06 / VRM-Addon-for-Blender

VRM Importer, Exporter and Utilities for Blender 2.93 to 4.2
https://vrm-addon-for-blender.info
MIT License
1.31k stars 143 forks source link

Feature request:要望 カスタムプロパティの使用を止めたい #63

Open FujiSunflower opened 3 years ago

FujiSunflower commented 3 years ago

Is your feature request related to a problem? Please describe.:バグじゃないけどアドオンの動作でなにか困ってることがあれば VRMの変数などをカスタムプロパティで保存していますが、今後のアップデートが複雑になるため使用を止めたいです。 プロジェクト全体に影響が発生する変更ですので、段階的な対応で良いと思います。

Describe the solution you'd like:どう動いてほしいかをどうぞ 代わりに下記のようにregister()でプロパティグループを用意して、そちらを呼び出すようにします。

class VRMProps(bpy.types.PropertyGroup):
    boo0: bpy.props.PointerProperty()
    boo1: bpy.props.PointerProperty()

def register(init_version: Any) -> None:
    bpy.types.Object.vrm_props =  bpy.props.PointerProperty(type=VRMProps)

def unregister() -> None:
    del bpy.types.Object.vrm_props

Additional context::なんかほかに書きたいことがあれば 実際の値自体の保存はまだテキストブロックに書き込む方式で良いと思います。 実際の処理の変更は行わず、プロパティグループとUIパネルだけ追加したものをプルリクで送るつもりです。

FujiSunflower commented 3 years ago

プロパティのgetterを使用してUIの一部から設定が出来るようになりました。 4

saturday06 commented 3 years ago

ご提案ありがとうございます。あと、SpringBoneとかVRMの拡張をjsonテキストで保存するのもやめたいんですよね・・・。

GWで時間が取れそうなので、そこで一気にやっちゃおうかなーとは思っています。

FujiSunflower commented 3 years ago

コンフリクトしていますが、参考程度にどうぞ #65