Incompatiblility with v11

[Txus5012]

Hi. Great module. Thanks!

In Foundry v11 doesn't seem to work, looks like an issue with the new PIXI.

[r-valyar]

+1 to have this nice module updated to v11 if possible! :)

[azarius74]

+1

[Bekradan]

+1

[messiahcide]

+1

[Melkkart]

+1

[Keladryel]

+1

[CodaBool]

@pplans do you plan on still maintaining this? I might have interest in forking if not.

[CodaBool]

have a working version. There are some console warnings about a sound file. I will try to resolve that but should be able to put out a PR to get this module fixed!

[Bekradan]

Fabulous.  Well,done.

Kind regards,Wayne

On Friday, September 1, 2023, 7:29 pm, CodaBool @.***> wrote:

have a working version. There are some console warnings about a sound file. I will try to resolve that but should be able to put out a PR to get this module fixed!

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

[pplans]

hi guys, yeah I plan to do so but have had troubles working on two parallel versions on my computer before and have no much time to digg it through. Am currently based on the stable version (10) until next release, you can make a PR I'll review it

[pplans]

Also, the sound file is an issue with the current state of the Foundry sound API which is kinda bad. I can't remember the details right now but it involved non cached memory for the sound boards, so each time you wanna play a sound it will try to load it. This process takes time and doesn't suit the sound for the motion tracker who needs high trigger precision

[CodaBool]

I have an implementation with AudioHelper, should have a PR out by end of today. We can discuss things once I have that made.

[CodaBool]

PR 🐱

[pplans]

PR is being reviewed, sorry for the delays, been sick & still at work :')

[Mizy78]

How can I install a working v11 version?

[CodaBool]

best to just wait for the PR to get merged

...but if you are feeling impatient the extra bug testing might be useful.

requires git installed on your server

1. install the v10 module
2. go to modules folder on your server for me this is .../Data/modules/
3. delete the motion_tracker folder
4. clone my fork git clone https://github.com/CodaBool/motion_tracker.git
5. make sure you have the module enabled and restart your server.
6. :sunglasses: should be using a v11 working version

if you run into any issues with the v11 version it'd be useful to know on that pr I opened.

[pplans]

Hi guys, the v11 migration should be online, thanks CodaBool for his efforts. Please let me know if we can close the issue

[rdefendi]

On a quick check the audio volume controls and the checkbox to ignore PCs don't seem to work. Other than that, it's great to have this module back!

[CodaBool]

Awesome, glad it's mostly working. I know why both those bugs are happening. Those should be straightforward fixes. I could probably put out a PR before the end of the month for those.

[pplans]

ok so am closing this issue then, we can make a second for the checkbox