FastScriptReload icon indicating copy to clipboard operation
FastScriptReload copied to clipboard

ImmersiveVRTools.Common.Runtime: UnityMainThreadDispatcher can't be initialized on worker thread

Open mollstam opened this issue 1 year ago • 2 comments

Just got hit by an exception where code tried to get the .gameObject property from a worker thread. It was during SafeInvoke(HotReloadFailed, changesAwaitingHotReload); triggering ProjectWindowReloadStatusIndicatorDrawer.OnHotReloadFailed which in turn accesses UnityMainThreadDispatcher.Instance which in SingletonBase<T> will try to access stuff that should only be accessed on the main thread in case there is no singleton instance. Sorry for oral account of the callstack but my debugger got detached when I was poking around and CBA to repro. 😅

Anyway, maybe the main thread dispatcher needs to eaglery be inited from the main thread (RuntimeInitializeOnLoad?) if its means to be accessed from a worker, or just use another singleton implementation that doesn't hit Unity's API.

mollstam avatar Aug 06 '24 04:08 mollstam

Thanks - that could be an option. It may be a while before I get anywhere close to that as it doesn't happen to often (I think?). Feel free to submit PR if you've got a solution in mind already

handzlikchris avatar Aug 06 '24 06:08 handzlikchris

I will if I keep running into it, so far not annoying enough :)

On Tue, 6 Aug 2024 at 08:06, Chris Handzlik @.***> wrote:

Thanks - that could be an option. It may be a while before I get anywhere close to that as it doesn't happen to often (I think?). Feel free to submit PR if you've got a solution in mind already

— Reply to this email directly, view it on GitHub https://github.com/handzlikchris/FastScriptReload/issues/134#issuecomment-2270455605, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC6ELRVTHNWLMQPZQT5IJ3ZQBRWRAVCNFSM6AAAAABMBO7U72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZQGQ2TKNRQGU . You are receiving this because you authored the thread.Message ID: @.***>

mollstam avatar Aug 07 '24 05:08 mollstam