NodeGraph.csのownerBehaviourObjectプロパティで不要なGC Allocが発生する

Here is a forum for reporting failure to Arbor developer.
Please read the forum rules carefully and enter the information necessary to confirm the problem.

Note:
  • Please read the "Update Guide" carefully when updating Arbor.
  • The bugs that we already know are summarized in "Known Issues", so please check once.
  • Please check ReleaseNote and use the latest version if possible before reporting a bug.
  • Unity beta and alpha versions are not supported. Please use the official release version.
  • In principle, defects caused by the introduction of other assets are not supported. If you find a defect, please report it after isolating which asset is the problem.

ここは、Arbor開発者へ不具合を報告するためのフォーラムです。
フォーラムルールをよく読み、不具合確認に必要な情報を記載するようお願いします。

注意点:
  • Arborを更新する際は「アップデートガイド」をよく読み行ってください。
  • 既に把握している不具合は「既知の問題」にまとめてありますので一度確認してみてください。
  • 不具合報告をする前にReleaseNoteを確認し、なるべく最新バージョンを使用して下さい。
  • Unityのベータ版およびアルファ版はサポート対象外です。正式リリース版をご利用ください。
  • 他アセット導入による不具合は原則サポート対象外です。不具合を見つけた場合はどのアセットの問題であるか切り分けた上でのご報告をお願いいたします。

Forum rules
The items necessary for reporting are as follows.
  • OS(Required)
  • Unity version(Required)
  • Arbor version(Required)
  • How to reproduce(Required)
    The reproduction method should be described accurately so that anyone can read the same procedure.
    If you do not know the exact reproduction procedure, as long as you can remember it, so please tell us what you were doing immediately before and the last time you edited it.
  • Build platform
  • Reproduction project (Minimum configuration not including Arbor and other assets) or video

Note:
  • Please read the "Update Guide" carefully when updating Arbor.
  • The bugs that we already know are summarized in "Known Issues", so please check once.
  • Please check ReleaseNote and use the latest version if possible before reporting a bug.
  • Unity beta and alpha versions are not supported. Please use the official release version.
  • In principle, defects caused by the introduction of other assets are not supported. If you find a defect, please report it after isolating which asset is the problem.

報告に必要な項目は以下の通りです。
  • OS(必須)
  • Unityバージョン(必須)
  • Arborバージョン(必須)
  • 再現方法(必須)
    再現方法については、誰が読んでも同様の手順を行えるよう正確に記述するようにお願いいたします。
    もし正確な再現手順が不明な場合、覚えている限りで構いませんので直前に行っていたことや最後に編集した時のことを教えてください。
  • ビルドプラットフォーム
  • 再現プロジェクト(Arborや他アセットなどを含まない最小構成)もしくは動画

注意点:
  • Arborを更新する際は「アップデートガイド」をよく読み行ってください。
  • 既に把握している不具合は「既知の問題」にまとめてありますので一度確認してみてください。
  • 不具合報告をする前にReleaseNoteを確認し、なるべく最新バージョンを使用して下さい。
  • Unityのベータ版およびアルファ版はサポート対象外です。正式リリース版をご利用ください。
  • 他アセット導入による不具合は原則サポート対象外です。不具合を見つけた場合はどのアセットの問題であるか切り分けた上でのご報告をお願いいたします。
ooee

NodeGraph.csのownerBehaviourObjectプロパティで不要なGC Allocが発生する

Post by ooee »

いつもお世話になっています。

不具合ではないのですが、不要なGC Allocが発生する部分に遭遇したので、ご報告いたします。

OS: Windows10
Unityバージョン: 2021.3.16f1
Arborバージョン: 3.9.2
再現方法:
NodeGraph.csのownerBahaviourObjectプロパティの336~342行目、
if (behaviour != null && !behaviour.attachedNode)
{
behaviour.delayAttachToNode += () =>
{
currentRootGraph.ChangedGraphTree();
};
}
で、ローカル変数をキャプチャしているため、このif文内が実行されなくてもGC Allocが発生しました。
参考:【Unity】ローカル変数をキャプチャする場合、その処理が実行されなくても GC Alloc は発生する https://baba-s.hatenablog.com/entry/2019/12/25/082000
User avatar
caitsithware
管理人
Posts: 495
Joined: 2015/08/17 12:41

Re: NodeGraph.csのownerBehaviourObjectプロパティで不要なGC Allocが発生する

Post by caitsithware »

ご報告ありがとうございます。
該当の箇所は今後の更新で修正いたします。

実行されない箇所でもラムダ式でのローカル変数のキャプチャによりGC Allocが発生するC#の仕様は把握しておりますが、
事前にすべて対処してリリースするのも難しいのでご報告していただけるのは助かります。
Locked