pub-solar/infra
6
1
Fork 3
Code Issues29 Pull requests9 Projects1 Releases Packages Wiki Activity Actions

nextcloud: add recognize app #285

Merged
teutat3s merged 2 commits from nextcloud-recognize-app into main 2025-02-09 17:59:09 +00:00
Conversation 5 Commits 2 Files changed 3 +8 -3
teutat3s commented 2024-12-19 16:46:08 +00:00
Owner

Adds https://github.com/nextcloud/recognize - "Smart media tagging and face recognition with on-premises machine learning models."

Adds https://github.com/nextcloud/recognize - "Smart media tagging and face recognition with on-premises machine learning models."
teutat3s added 4 commits 2024-12-19 16:46:08 +00:00 
based on PR: https://github.com/NixOS/nixpkgs/pull/333545
Flake lock file updates:

• Updated input 'unstable':
    'github:nixos/nixpkgs/3566ab7246670a43abd2ffa913cc62dad9cdf7d5' (2024-12-13)
  → 'github:nixos/nixpkgs/d3c42f187194c26d9f0309a8ecc469d6c878ce33' (2024-12-17)
requested reviews from b12f, hensoko, axeman 2024-12-19 16:46:23 +00:00
teutat3s commented 2025-01-20 19:23:19 +00:00
Author
Owner

An alternative way I found online that would allow us to keep nextcloud app autoUpdates enabled:

sudo mv /path/to/nextcloud/store-apps/recognize ~/recognize.bak
sudo cp -r "$(readlink result)" /path/to/nextcloud/store-apps/recognize
sudo chown -R nextcloud:nextloud /path/to/nextcloud/store-apps/recognize

Also this could get merged after updating our infra to NixOS 24.11 in to use the backported version of recognize instead of using the unstable overlay.

An alternative way I [found online](https://discourse.nixos.org/t/nextcloud-with-declarative-apps-memories-recognize-unable-to-download-models-for-recognize-app/41726/21) that would allow us to keep nextcloud app autoUpdates enabled: ``` sudo mv /path/to/nextcloud/store-apps/recognize ~/recognize.bak sudo cp -r "$(readlink result)" /path/to/nextcloud/store-apps/recognize sudo chown -R nextcloud:nextloud /path/to/nextcloud/store-apps/recognize ``` Also this could get merged after updating our infra to NixOS 24.11 in #278 to use the [backported version of recognize](https://github.com/NixOS/nixpkgs/pull/366241) instead of using the unstable overlay.
b12f commented 2025-01-20 20:43:00 +00:00
Owner

We could package all our necessary apps as nix packages and have them updated via nvfetcher, tho that's a llot more involved

We could package all our necessary apps as nix packages and have them updated via nvfetcher, tho that's a llot more involved
teutat3s commented 2025-01-30 15:42:16 +00:00
Author
Owner

The Nextcloud Apps auto update has broken our Nextcloud once in the past. I would be in favor of having all Nextcloud Apps in nix.
From a quick review, the following Apps would need to be packaged:

integration_deepl
quota_warning
The Nextcloud Apps auto update has broken our Nextcloud once in the past. I would be in favor of having all Nextcloud Apps in `nix`. From a quick review, the following Apps would need to be packaged: ``` integration_deepl quota_warning ```
hensoko commented 2025-01-30 17:13:47 +00:00
Owner

We can also use fetchzip instead of packaging:
Example: https://git.pub.solar/hensoko/nixos-config/src/branch/main/hosts/chonk/nextcloud-apps.nix#L21

We can also use fetchzip instead of packaging: Example: https://git.pub.solar/hensoko/nixos-config/src/branch/main/hosts/chonk/nextcloud-apps.nix#L21
❤️ 1
teutat3s force-pushed nextcloud-recognize-app from c4aa25f711 to 0595124a1e 2025-02-06 00:39:21 +00:00 Compare
teutat3s force-pushed nextcloud-recognize-app from 0595124a1e to 0d7b88af11 2025-02-06 00:43:39 +00:00 Compare
teutat3s changed title from nextcloud: add recognize app, disable apps autoUpdate to nextcloud: add recognize app 2025-02-06 00:44:30 +00:00
teutat3s commented 2025-02-06 00:49:59 +00:00
Author
Owner

Found out in the end that it's totally possible to have both apps from the Nextcloud app store installed via UI and apps installed via nix. This is ready for review.

Found out in the end that it's totally possible to have both apps from the Nextcloud app store installed via UI and apps installed via nix. This is ready for review.
teutat3s force-pushed nextcloud-recognize-app from 0d7b88af11 to 4c6911ea6f 2025-02-07 14:03:52 +00:00 Compare
hensoko approved these changes 2025-02-09 17:57:38 +00:00
hensoko left a comment
Owner

LGTM

LGTM
teutat3s merged commit c03f0c5a2a into main 2025-02-09 17:59:09 +00:00
teutat3s deleted branch nextcloud-recognize-app 2025-02-09 17:59:09 +00:00
Sign in to join this conversation.
No reviewers
b12f
axeman
hensoko
No labels
Bug
Docs
Feature request
Good for newcomers
Infra
Privacy
Refactoring or migration
Security
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: pub-solar/infra#285
No description provided.
Delete branch "nextcloud-recognize-app"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?