Video URL not show when following Peertube - action: resolved - awaiting verification. #63

Hallo again :-)
If you are busy these days, just let me know. Still enjoying exploring!
I have a few notes, but this one seems worthy of making an issue now, unless I'm doing something wrong or not understanding.
It happens that other people in the thread can click and see the vid, but my follow doesn't have the link.

image

Could you click the cog wheel, 'View Source' and paste the json (or send to my email)?

Peertube keeps changing the way their non-standard activities are structured, which means every other project has to change something to make shit work again whenever they do. I haven't bothered to keep up since I retired and I'm not following any peertube channels currently.

Though I will also say that we generally only pick out mpegs which come from the peertube server itself and skip the torrents. And that server might have gotten bogged. But most likely, they've changed their attachment structures again.

I put the source of one in my pastebin: https://pastebin.kalum.ca/?d62886db0b2a6901#B2rvNtSZqC2Et9tkcVNjM3HXSaW3vWskcJWmvWhe3GCi
Hey, I don't want to be a bother - I'm over 65 myself and I stopped maintaining a couple of things - one over 20 years - old a couple of years ago (a gpl canadian payroll). I have like 2-3 UX notes I've been holding back and one feature request, b/c I don't want to overwhelm you. In general, I really like this better than masto - and also, your earlier work friendica, which I trialed under that project name some years ago.

Yikes. Yeah, looks like they messed with the Activity structure again. I'll see what I can do.

We normally try and find a low resolution video to display inline, and the size choices have changed. There is no longer a video with less than 500px width. The smallest one is now 1280 (which is larger than our app window). It scales fine, but now we will choose any video less than 1500px wide for display [edit: even if there are smaller choices available - fixing that will require more effort].

Anyway, I've got a fix. Will work this into a release and push in the next few days.

Oh! No!
I didn't expect that - that's just more cpu and overhead for everyone!
Just put in the https:link to the original video. Surely that's enough?
ffmpeg to 3gp or w/e is too much cpu for such an often thing. I think.

Peertube does a lot of transcoding. It used to provide a number of different resolutions (7 or 8 if I recall), but now they only support 2 and the smallest is a fair bit larger than before.

We're not transcoding anything, just trying to find the best resolution for our needs - or whatever they've got if we can't. The first part is working with peertube again. I'll get to the second part as time permits. I'm not too concerned.

I know PT does a lot of transcoding - trust me! I now have:

Group=peertube
Nice=10 
MemoryHigh=1G 
MemoryMax=1.5G 

But what I'm saying is why are you doing it? Just put in the link. Any transcoding is our problem. No need to embed - is that what you're doing? Just link it.
I'm sorry, I'm not understanding here.

We provide a link to the "best" resolution that Peertube provides. There are a number of links and we choose the best one for our application. The best one is a bit bigger than before and I wasn't finding it because I was looking for one that was under 500px wide - and those are no longer provided. Sure, we could just provide a link, and not the actual playable video, but that's sooo 2002. I will ensure the link is added as well as the video element. Will polish that up when I get the time. In the meantime, the videos and associated "poster" image are now provided inline where they weren't previously.

Ok - thank you! Maybe it's a part of the spec? I don't know. And I'm soooo b. 1960 so... :-)

v26.4.25 was released yesterday.

Thanks so much!
I'm just waiting for YUNOhosts tooling to see the PR