unbox icon indicating copy to clipboard operation
unbox copied to clipboard

[djay pro - Trouble Connecting - Resolved (Album Art Update)]

Open clarityam opened this issue 1 year ago • 23 comments

OS: Windows 11 Pro - 23H2 - 22631.3085 djay pro: 5.1.77.18.0 Unbox: Latest (90ab1b8)

App initialization works as expected for Unbox and djay pro, unfortunately no connection is being established between djay pro and unbox, there was a recent update to add Apple Music to djay pro which may have caused some changes in communication.

Thanks!

clarityam avatar Feb 06 '24 16:02 clarityam

Thanks for sending this over and sorry that you're running into issues here, will take a look at this today and get a new release out.

erikrichardlarson avatar Feb 06 '24 16:02 erikrichardlarson

Absolutely! After VSCode'ing your src, it looks like my instance of djay may not be spawning the instance of NowPlaying.txt that is required to pull the track information. Not sure if this is due to the recent update or it likely could be an isolated issue with my machine

clarityam avatar Feb 06 '24 17:02 clarityam

This looks like something related to the Windows version, used procmon to check for file writes and didn't see track data being written to any files. Going to keep looking into this and see what else can be done, in the meantime I'll refund your subscription.

erikrichardlarson avatar Feb 07 '24 02:02 erikrichardlarson

No worries on this at all! Also no need to refund anything, I am completely happy with supporting you!

clarityam avatar Feb 07 '24 02:02 clarityam

This looks like something related to the Windows version used procmon to check for file writes and didn't see track data being written to any files. Going to keep looking into this and see what else can be done, in the meantime I'll refund your subscription.

I was able to replicate the same thing you described, I tried switching tracks against different platforms for music just to make sure it was not a random fluke but I was unable to find any process that would spawn the NowPlaying file (using procmon as well). My friend utilizing a Mac couldn't seem to locate the file on his side either but we are yet to test using Unbox on the device, so it may be just an isolated Windows issue. I have in the meantime put in a request to Algoriddim to see if that file is still being created by a process in the software.

Update: Tested it on Mac with the same results, I believe that Djay Pro is not writing the file meant to be read, your code looks fine to me, it's how I would've done the implementation.

clarityam avatar Feb 07 '24 16:02 clarityam

Good to know, I was actually able to parse the WAL file to get the artist / track / isrc as records are written to the db so I'll update the poller to use this code this week. Will keep you updated here.

erikrichardlarson avatar Feb 09 '24 02:02 erikrichardlarson

Fantastic! I will be looking forward to it!

clarityam avatar Feb 09 '24 20:02 clarityam

Just pushed a new release that uses a combination of the WAL file and db to get the history updates on Windows, let me know if you run into any issues: https://github.com/erikrichardlarson/unbox/releases/download/11.3/unbox-win.zip

erikrichardlarson avatar Feb 10 '24 03:02 erikrichardlarson

At the moment, the client reports a successful connection, although I am unable to utilize the local overlay (http://localhost:8001/unbox_overlay.html) on my current computer, it reports "Cannot GET /unbox_overlay.html", in addition, album art (http://localhost:8001/album_art.html) shows nothing.. I have djay installed on another device as well so I am going to test that and get back to you.

clarityam avatar Feb 10 '24 04:02 clarityam

Are you accessing the overlay html on the same machine that unbox is running on?

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

Indeed. I am attempting to use the localhost URL, I would test the external but it got assigned 192.168.1.x (I am assuming because of the Virtual Machine adapters in windows)

clarityam avatar Feb 10 '24 04:02 clarityam

Hmm does the external also fail?

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

Let me spin up a VM real quick and I'll take a look. I tested internal on a different computer and had the same issue.

clarityam avatar Feb 10 '24 04:02 clarityam

Album art isn't working yet btw, will get an update out for that and will work when an isrc can be resolved.

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

No worries. I just tested it on a VM (external IP 192.168.x.x) and I'm having the same issue with "Cannot GET /unbox_overlay.html"

clarityam avatar Feb 10 '24 04:02 clarityam

Ah try changing the overlay color to see if that copies over the file needed.

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

Haha! Funny enough, that is the solution, works perfectly!

clarityam avatar Feb 10 '24 04:02 clarityam

Great glad it's working, I'll fix that too. There is another issue where the first track played isn't picked up sometimes so might need a dummy first track before starting actual set.

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

Ah yes, I know that's certainly an issue if unbox is started after Djay as well. (Which is not the way it's to be used anyway) It has that issue til another track is loaded on a deck then it resolves that change in the db. Overall, excellent work though!

clarityam avatar Feb 10 '24 04:02 clarityam

Thanks! Where are you playing your tracks from btw? Beatport provides the isrc, but not sure if other track sources do

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

I have a few to experiment with, unfortunately, Beatport isn't one of them, however, I have Apple Music, Tidal, and Soundcloud+ available to test! (I've tried Apple Music so far and it reports the Artist and the Track perfectly, Tidal resolves same information, Soundcloud as well!)

clarityam avatar Feb 10 '24 04:02 clarityam

Awesome, will post here when the album art release is up

erikrichardlarson avatar Feb 10 '24 04:02 erikrichardlarson

Perfect! Thank you for your hard work on this! It's a fantastic app! (It certainly makes me want to dive more into electron too lol)

clarityam avatar Feb 10 '24 04:02 clarityam