-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dependency hell: Call to undefined method Labeled::features() #676
Comments
I also have the same error message with the same configuration. root@f-------:/# occ recognize:reset-face-clusters DualTreeBall.php: 107: foreach ($dataset->features() as $column => $values) { The foreach construct provides an easy way to iterate over arrays. foreach works only on arrays and objects, and will issue an error when you try to use it on a variable with a different data type or an uninitialized variable Do you think it's a variable declaration problem? |
What other apps do you have installed? |
Activities for shared file downloads 1.15.0 |
yep, it's likely a conflict with a different app that's loading an older version of rubixML (a dependency) |
The fix for this is to use mozart in this project (note to self) |
Good call. Disabling |
Indeed after disabling Suspicious Login 4.3.0 it no longer generates an error. THANKS |
Still got errors after disabling Suspicious Login :( |
What other plugins do you have enabled @Chuckame? |
Yes, face recognition, I'll try disabling it then. Is there any way to not disable other plugins? |
I also had to disable current app setup:
|
see #676 Signed-off-by: Marcel Klehr <[email protected]>
…ndency hell see #676 Signed-off-by: Marcel Klehr <[email protected]>
using mozart was more trouble than it was worth apparently. Sorry, this is a won't fix. |
@marcelklehr Still an issue on 3.6.3 can you please reopen this ? |
Not the best issue I've posted, but think it covers what's needed: nextcloud/suspicious_login#780 Feel free to 👍🏼 |
@enoch85 this is still present on NC 26.0.0 running rec app 3.7.0 |
@AndyXheli Actually don't use it myself, so I don't know. But didn't get any complaints from our VM users for a while so I guess it works? Feel free to try it out. |
@marcelklehr Since the error is still present and no fix yet. Can we please reopened this issue ? My server logs are geting spammed with this. |
@AndyXheli As I've already stated: The error comes from an incompatibility between two apps. This app and some other app that uses RubixML. There is no other solution than disabling one of the two apps. If you have one, feel free to open a pull request to fix this app. Until then, I'm leaving this closed, but pinned, to make people aware that this app is incompatible with others. |
Thank you very much! |
Just adding, this is still an issue on Nextcloud 27 installed with AIO. Disabling
|
Having this issue. Disabling Version `app:list`
|
I've been trying to fix this incompatibility. If you are interested you can test the beta release of this effort: v6.1.0-beta.1 (release) You can install it by switching your nextcloud release mode to "beta" and then updating the recognize app to v6.1.0-beta.1. After that, you can switch back to the release mode you were on before. If you encounter problems, feel free to open a new issue. |
v6.1.0-beta.1, nextcloud version 28.0.1 suspicious login disable... |
found another apps: Mail v 3.5.4 used rubix... disable this app and error gone |
Unfortunately, this is more loss of functionality than suspicious_login ... |
This comment was marked as off-topic.
This comment was marked as off-topic.
The whole problem is that one common autoloader is used for all applications and then a dependency conflict occurs |
@Monoboy4ik Thank you for reporting! I'm looking into it |
Thank you too! |
The new beta is out. Shouldn't require disabling neither mail nor suspicious_login |
Testing |
thanks, everything seems to be working so far! |
@marcelklehr beta seems to be running with no issues on NC 28.0.2 rc4 with suspicious app enabled |
@marcelklehr is the beta fix already in a stable version? |
Yes, it's in the latest version |
Then the README "Requirements" Section should be adjust that there are no more compatible issues!? |
Ah, yes. Thanks for the heads up. Will fix the README :) |
Which version of recognize are you using?
3.5.0
Enabled Modes
Face recognition
TensorFlow mode
WASM mode
Which Nextcloud version do you have installed?
25.0.3
Which Operating system do you have installed?
Debian 11
Which Docker container are you using to run Nextcloud? (if applicable)
How much RAM does your server have?
8GB
What processor Architecture does your CPU have?
ARM64
Describe the Bug
Running
occ recognize:cluster-faces
results in an unhandled exception.Expected Behavior
Cluster faces without throwing an error.
To Reproduce
Try running
occ recognize:cluster-faces
Debug log
The text was updated successfully, but these errors were encountered: