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

Subtitles on tv4 #229

Open
4 of 5 tasks
ingemarh opened this issue Oct 2, 2024 · 9 comments · May be fixed by #238
Open
4 of 5 tasks

Subtitles on tv4 #229

ingemarh opened this issue Oct 2, 2024 · 9 comments · May be fixed by #238

Comments

@ingemarh
Copy link

ingemarh commented Oct 2, 2024

Innan du skickar in hjälpförfrågan

Länk till video du behöver hjälp med

https://www.tv4play.se/video/86e500670073968a828f/avsnitt-2-impasto

Kryssa i den här rutan om alla klipp på sidan har slutat fungera

  • Alla klipp på sidan har slutat fungera

Beskriv vad som inte fungerar

First try of this extension. The video goes fine, gives me Impasto.mp4. As I understand, I should just change the output name to Impasto.srt in the gui to get the subtitles. However, this command fails with:
[out#0/srt @ 0x5568bba92280] Output file does not contain any stream
Error opening output file Impasto.srt.
Error opening output files: Invalid argument
Tried also with the ffmpeg-builds, same error.
Running on linux and Firefox.
svtplay-dl gives me both .mp4 and .srt, but it seems the token have much shorter expiry time nowadays, so a bit tedious...

@robban00
Copy link

Vet ingen lösning på texter på varken tv4play och svtplay :(

@Lunarwalker60
Copy link

Lunarwalker60 commented Oct 16, 2024 via email

@robban00
Copy link

Vad är problemet? Mina automatiska rutiner är inte de enklaste men fungerar klockrent både för SVT & TV4 så lösningar finns.--Send from my Android phone with mail.com Mail. Vänligen ursäkta min korthet. Den 2024-10-16, 02:25 robban00 @.> skrev: Vet ingen lösning på texter på varken tv4play och svtplay :( —Svara direkt på detta mail, se det på GitHub, eller avsluta prenumerationen. Du får detta eftersom du prenumererar på den här tråden. Meddelande-ID: @.>

Finns det någon förklaring på ett lätt sett?

@Lunarwalker60
Copy link

Lunarwalker60 commented Oct 16, 2024 via email

@Lunarwalker60
Copy link

Jag har tittat på länkarna för ett specifikt program för att se hur man manuellt kan få fram en fungerande textlänk.
Kommandot från Privatkopiera:
ffmpeg -i "https://vod.streaming.a2d.tv/45a0fa7b-596c-407c-9b01-6b4803e286c0/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646.ism/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646-video=800000.m3u8" -i "https://vod.streaming.a2d.tv/45a0fa7b-596c-407c-9b01-6b4803e286c0/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646.ism/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646-audio=128000.m3u8" -c:v copy -c:a copy "Fauve.mkv"

Kommando för undertext:
ffmpeg -i "https://vod.streaming.a2d.tv/45a0fa7b-596c-407c-9b01-6b4803e286c0/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646.ism/hls/9e087fa0-7690-11ef-9b81-fdf9cd1e5244_20857646-textstream_swe=1000.vtt" "Fauve.srt"

Om Du tittar efter så kan Du lista ut hur Du manuellt ska fixa till ett undertext-kommando.
OBS att länkarna ovan inte kommer att fungera direkt hos dig - de bygger på cookies hos mig.

@Lunarwalker60
Copy link

Jag har skrivit en snutt i Excel där man klistrar in kommandot från Privatkopiera, klickar på en knapp och får kommandot för att tanka hem texten. Klicka på en annan knapp så lägger kommandot sig i urklipp, likadant som Privatkopiera gör.
Detta funkar för TV4 Play men det går inte att bifoga Excel-makrofiler här.

@Lunarwalker60
Copy link

Lägg en kommentar här om det är intressant med Excel-funktionen för TV4-undertexter så hittar vi på ett sätt att föra över den.

@Phiplex
Copy link

Phiplex commented Oct 17, 2024

Enklast är väl om du bara lägger själva Excels VBA kod här - och inte hela .xlsm filen. Så kan den som vill använda din kod lägga in den själv i Excel mha VBE (Visual Basic Editor) som finns i Excel.

@Lunarwalker60
Copy link

Alla som behöver funktionen kanske inte är bekanta med VBA & VBE men jag gör ett försök att förklara.
MakeSRT

Öppna en ny Excel-arbetsbok

Öppna VBE med Alt-F11

Markera Thisworkbook i projektutforskaren

Klistra in:

Private Sub Workbook_Open()
MakeSRT.Show
End Sub

Infoga ett UserForm och namnge det MakeSRT

Välj Visa objekt
Klistra in två väl tilltagna textrutor, TextBox1 & TextBox2,
båda med egenskapen Multiline = True

Klistra in tre Cmdbuttons med namn; cmdText, cmdCopyLink, cmdQuit

Välj Visa kod

Klistra in koden mellan de streckade linjerna:

DefStr S
DefInt N

Const sQt = """"

Private Sub TextBox1_Change()
If InStr(TextBox1, "-c:v copy -c:a copy") Then
cmdText.Enabled = True
End If
End Sub

Private Sub cmdText_Click()
Dim np1, np2, np3, np4
Dim sPKLink, sSRTLink
Dim sTmp, sFind

sPKLink = TextBox1
np1 = InStr(sPKLink, ".ism/")
np2 = InStr(sPKLink, "-video")
sFind = "-c:v copy -c:a copy "
np3 = InStr(sPKLink, sFind)
If np3 > np2 Then
np3 = np3 + Len(sFind)
End If

If (np1 > 0) And (np2 > np1) Then
np1 = np1 + 4
sSRTLink = Left(sPKLink, np1) & "hls/"

  sTmp = Mid(sPKLink, np1 + 1, (np2 - np1) - 1)

  sSRTLink = sSRTLink & sTmp & "-textstream_swe=1000.vtt" & sQt & " "
  If np3 > np2 Then
     sTmp = Mid(sPKLink, np3)
     np4 = InStr(sTmp, ".")
     If np4 > 0 Then
        sTmp = Left(sTmp, np4) & "srt" & sQt
        sSRTLink = sSRTLink & sTmp
     End If
  End If

End If
TextBox2 = sSRTLink
cmdCopyLink.Enabled = True

End Sub

Private Sub cmdCopyLink_Click()
Dim MData As DataObject

Set MData = New DataObject
MData.SetText TextBox2
MData.PutInClipboard

End Sub

Private Sub cmdQuit_Click()
End
End Sub


Spara arbetsboken, stäng den och öppna den sen igen.
Om allt blivit rätt så visas formuläret och Du kan
klistra in kommandot från Privatkopiera i TextBox1
och klicka på knappen "Ta fram undertext-länk"
När kommandot visas i TextBox2 så klickar Du på
knappen "Kopiera undertext-kommando" så ligge det i urklippp
på samma vis som video-kommandot från Privatkopiera gjorde.

Lycka till

@agreedSkiing agreedSkiing linked a pull request Dec 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants