#I would like an evaluation from people more experienced in GO.
raw cloud
fair cargo
don’t need internal/ nor cmd/ and does need a license
raw cloud
don’t need internal/ nor cmd/ and does need a license
There's a license in the main directory, but why not in "cmd" and "internal"? Just for organizational purposes.
tall peak
Hi,
can you explain me the code related to this line ?
https://github.com/gabriel-logan/yt-dlp/blob/8f568a30d316caa94b5845b7328f27137f56f6f3/server/internal/api/video.go#L58
fair cargo
Hi,
can you explain me the code related to this line ?
https://github.com/gabr...
it blocks until there is a slot (1/2 of cpus slots)
let's say that's 4, then first 4 can write the channel the other block until one of the 4 reads in defer
raw cloud
Hi,
can you explain me the code related to this line ?
https://github.com/gabr...
It's like a queue
To avoid overloading the processor, because yt dlt is resource-intensive.
fair cargo
´cept it should not be cpu intensive, it’s all IOs, or why not?
raw cloud
´cept it should not be cpu intensive, it’s all IOs, or why not?
My Go app itself doesn't require much CPU, but it calls a Python binary that does use CPU.
fair cargo
ah so it’s not the download you are guarding but the post processing
convert it to go :)?
raw cloud
Yes
tall peak
It's like a queue
You may be interested in this semaphore https://pkg.go.dev/golang.org/x/sync/semaphore