Skip to content
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

BUG PhotoFace-Patch #4256

Open
MartinKuhl opened this issue Oct 30, 2024 · 9 comments
Open

BUG PhotoFace-Patch #4256

MartinKuhl opened this issue Oct 30, 2024 · 9 comments
Labels

Comments

@MartinKuhl
Copy link

Install ENV:

CPU: Intel Atom
NIC: SKY2
RR version: latest 24.10.3

DSM:

model: 3622xs+
version: 7.2.1 Patch 5

Issue:

The PhotoFace-Patch is not enabling the face recognition in DSM. All tries to recognize the faces ending in timeouts.
If I directly execute the patch from here: https://github.com/jinlife/Synology_Photos_Face_Patch/blob/main/bin/patch.sh
it is directly working. Is there a special order how to install the DSM Photo app and installing the patch from the loader after the DSM-app installation?

@wjz304
Copy link
Contributor

wjz304 commented Nov 1, 2024

加载程序选则addon,启动DSM后安装 Photo ,重启DSM

@MartinKuhl
Copy link
Author

I alreasy installed the addon but it isn't working

@wjz304
Copy link
Contributor

wjz304 commented Nov 4, 2024

I tested it and found that there is a bug in the addon.
Since Synology Photos is installed in the storage pool, the path where it is located will only exist after the application is started.
And the addon is run too early, resulting in the path where the photos library does not exist during runtime.

@wjz304
Copy link
Contributor

wjz304 commented Nov 4, 2024

This problem is currently difficult to solve.

@wjz304 wjz304 added the Waiting label Nov 4, 2024
@MartinKuhl
Copy link
Author

Looks Like arc loader fixed this issue in one of the last updates. Just FYI maybe you can adopt this solution.

@wjz304
Copy link
Contributor

wjz304 commented Nov 4, 2024

The changes of arc don't make sense, but I did find a solution.

@wjz304
Copy link
Contributor

wjz304 commented Nov 5, 2024

@AuxXxilium
Copy link

The changes of arc don't make sense, but I did find a solution.

they make sense, because the load point of service was wrong: if we change it back toAfter=multi-user.target it is working like it should

@wjz304
Copy link
Contributor

wjz304 commented Nov 7, 2024

The changes of arc don't make sense, but I did find a solution.

they make sense, because the load point of service was wrong: if we change it back toAfter=multi-user.target it is working like it should

Yes, but that's not strict, the photosfacepatch service should be run after the storage pool is loaded (theoretically, before the photo app is started).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants