Request: Please update to support Nextcloud 22.1.0 about recognize

[marcelklehr]

@isaacolsen94 see #16

[bonswouar]

I can paste the entire output of the command if that will help?

No need, the weird thing is that it can execute occ command and not node binary.. But I guess it just needs to be able to execute php for that.
There wouldn't be some execution protection or something on your volume/config/something? Maybe you should ask Unraid community (or wait to see if someone here might be able to help more)

[KruyKaze]

No worries, I'm waiting for next release anyway to fix the maxarg issue so I can test .

…

On Wed, Aug 11, 2021, 16:26 bonswouar ***@***.***> wrote: @KruyKaze <https://github.com/KruyKaze> sorry we kinda hijacked your thread ;) Does my first message answer to your problem or you have another issue? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#45 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAARFYKQI4IBSZ3AHIFMD73T4L2JVANCNFSM5B22RF2A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email> .

[isaacolsen94]

Also running 22.1.0 and I'm having issues. A new install however so I can't say if it was working before. But support for the latest install would be awesome! Thank you!

[bonswouar]

It's already been changed in a previous commit, it will be ok when the new release will be ready (hopefully soon!).
Meanwhile latest package should still work if you install it manually, but doesn't it just appear as "untested apps" in Apps list anyway?

[isaacolsen94]

I'm new to Nextcloud so I just did a brand new install of version 22.1.0 and installed Recognize. From the settings page it tells me I have 0 of 0 pictures ready to go. I ran the command manually and I get this error:

Running array (
  0 => '/config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64',
  1 => '/config/www/nextcloud/apps/recognize/src/classifier.js',
  2 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.34.jpg',
  3 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.35.jpg',
  4 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.36.jpg',
  5 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.37.jpg',
  6 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.38.jpg',
  7 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.42.jpg',
  8 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.43.jpg',
  9 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-16 20.48.54-1.jpg',
  10 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-16 20.48.54.jpg',
  11 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-16 20.48.57-1.jpg',
  12 => '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-16 20.48.57.jpg',
)

Classifier process output: sh: exec: line 1: /config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64: not found

Classifier process output: sh: exec: line 1: /config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64: not found

So I checked for said file:

I assumed it was because of 22.1.0 incompatibility, is it actually something else?

[bonswouar]

@isaacolsen94 Weird, but it seems to be unrelated to NC 22.
Could you try running the nodejs command yourself?
/config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64 '/config/www/nextcloud/apps/recognize/src/classifier.js' '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.34.jpg'

[isaacolsen94]

@isaacolsen94 Weird, but it seems to be unrelated to NC 22.
Could you try running the nodejs command yourself?
/config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64 '/config/www/nextcloud/apps/recognize/src/classifier.js' '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.34.jpg'

How would one do this? Sorry not very familiar with nodejs

[bonswouar]

How would one do this? Sorry not very familiar with nodejs

@isaacolsen94 I mean, just execute it, this is the command, nodejs binary is the first part (/config/www/nextcloud/apps/recognize/bin/node-v14.9.0-linux-x64)
If it exists I don't see why you'd have this error

[isaacolsen94]

I thought it might be a permissions error so changed node-v14* to 777 and still unable to find it

[bonswouar]

@isaacolsen94 you forgot the ./ in ./node-v14.9.0-linux-x64 needed when executing executable in current folder (your server is not on Windows, I hope 😄 )
From there the full command would be : ./node-v14.9.0-linux-x64 '/config/www/nextcloud/apps/recognize/src/classifier.js' '/data/iolsen/files/Photos/Isaacs Phone - Test/2020-04-13 08.19.34.jpg'

[isaacolsen94]

I thought it might be a permissions error so changed node-v14* to 777 and still unable to find it

[bonswouar]

Hm I'm lost haha.
What OS is your server running on?
Yeah I thought about permissions, but you should have a permission error then... Plus right now you're testing as root
EDIT : Maybe try running it as user abc anyway (sudo -u abc), it guess that's your webserver's user right? occ command should also be run from this user

[isaacolsen94]

I'm running Unraid 6.9.2. And my user inside Nextcloud is 'iolsen' but I tried both and got this:

[bonswouar]

Unfortunately I don't know much about Unraid, are you executing it from nextcloud docker's console right now?

[isaacolsen94]

Unfortunately I don't know much about Unraid, are you executing it from nextcloud docker's console right now?

Yes I am. When I run 'occ recognize:classify' it runs fine, just can't seem to see that file. I can paste the entire output of the command if that will help?

[bonswouar]

@KruyKaze sorry we kinda hijacked your thread ;)
Does my first message answer to your problem or you have another issue?

[marcelklehr]

v22 support is fixed in master, next release will support v22.