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

Updated the Synology instructions #867

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

LeRoid-hub
Copy link

In this RP i updated the Synlogy instruction for DSM 7.

docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
LeRoid-hub and others added 23 commits February 28, 2024 10:14
@Zelaf
Copy link

Zelaf commented Feb 28, 2024

Some input:

  • DLNA port specification can be ignored under the port section. Instead add the note that it requires host and uses the DLNA port 1900 under the network mode to reduce redundancy.
  • This will not use hardware tanscoding, on my setup I used the built-in "Projects" function instead to use compose files. Without tanscoding most lower end Synologies will completely halt when transcoding starts. The Synology will not be responsive for several minutes and upwards of 20 minutes afterwards, at least from my experience on using it on my DS720+.
  • A recommendation about upgrading the RAM for devices with 2GB or less should be added. The UI will be painfully slow without it. After I put a 16 gig stick in mine it was very performant. This might also be resolvable by using an external SSD or nvme SSD volume, as supported by the newer ones.

I did wrote a guide I never got the time to PR a while back with tanscoding and security in mind. I took screenshots as well. We might be able to incorporate both of them in some way.

@JPVenson
Copy link
Member

@Zelaf

  • The DLNA port is irrelevant if you dont use host mode anyway which is required for the feature to work.
  • I think that all Hardware requirements are flat handled via the top disclaimer that its generally a bad idea to run a media server on something that is not at all capable of handing the expected demand.
    All those information, minimum RAM, GPU and disk speed are talked about in the provided link to the hardware selection guide and we dont want to fragment those information again.

@Zelaf
Copy link

Zelaf commented Feb 28, 2024

@Zelaf

* The DLNA port is irrelevant if you dont use host mode anyway which is required for the feature to work.

* I think that all Hardware requirements are flat handled via the top disclaimer that its generally a bad idea to run a media server on something that is not at all capable of handing the expected demand.
  All those information, minimum RAM, GPU and disk speed are talked about in the provided link to the hardware selection guide and we dont want to fragment those information again.

Very true! But there of moving the mention of the port 1900 under port settings and noting it down under Network Settings would, imo, streamline the reading as well as people who prefers to skim documentation could find mentions of the DLNA host requirement easier to cause less possible confusion.

As for the hardware limitations, reading over it again in full it should indeed cover it. Will certainly be better than what is shown now, which is nothing!

I do hope this will ease the process for the many Synology users. I'd still like to see what the maintainers thoughts on providing a compose guide would be and if that guide should perhaps be under a separate section of this page or perhaps even included in the hardware acceleration page? The process is similar and still handled through the GUI.

@JPVenson
Copy link
Member

Thats why there is a disclaimer at the very start of the proposed doc that links to the hardware guide.
Esp. the warnings with low hardware systems.

Our docs are quite extensive how to use compose files with hardware acceleration
https://jellyfin.org/docs/general/installation/container#with-hardware-acceleration
That is covered there well and good.

@felix920506
Copy link
Member

@LeRoid-hub is this ready for re-review?

docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md Outdated Show resolved Hide resolved
docs/general/installation/synology.md.orig Outdated Show resolved Hide resolved
docs/general/installation/synology.md.orig Outdated Show resolved Hide resolved
docs/general/installation/synology.md.orig Outdated Show resolved Hide resolved
@LeRoid-hub
Copy link
Author

It would be nice to get this merged i dont have so much time on my hand anymore.

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

Successfully merging this pull request may close these issues.

5 participants