Dantotsu icon indicating copy to clipboard operation
Dantotsu copied to clipboard

Add the ability to stream torrents video in app itself.

Open ARIF683 opened this issue 11 months ago • 29 comments

As of now, the app open the torrent video in different app, make it stream torrents in the app itself.

ARIF683 avatar Mar 28 '24 15:03 ARIF683

https://github.com/rebelonion/Dantotsu/assets/70383209/5191f8eb-4608-4252-99cc-c3015a8f427b

@AbandonedCart 😅

ARIF683 avatar Mar 28 '24 15:03 ARIF683

😢

ARIF683 avatar Mar 28 '24 16:03 ARIF683

Okk leave this now.

Can you add something like , accessing content of an extension. As of now only way to do it is through clicking wrong title.

ARIF683 avatar Mar 28 '24 16:03 ARIF683

I feel like that one sort of breaks the whole concept. If you're just looking through extensions directly, you wouldn't really need the rest of the app. Half the fun is stumbling through it.

AbandonedCart avatar Mar 28 '24 16:03 AbandonedCart

Cancel

ARIF683 avatar Mar 28 '24 16:03 ARIF683

Besides having to search in the other direction to link it back up to all the tracking and stuff.

AbandonedCart avatar Mar 28 '24 16:03 AbandonedCart

Let me think something else 😅

ARIF683 avatar Mar 28 '24 16:03 ARIF683

Sub/dub toggle for gogo, animepahe and many more extension aren't implemented.

ARIF683 avatar Mar 28 '24 16:03 ARIF683

I found a library that is supposed to work with exo, so I'm doing some research

AbandonedCart avatar Mar 28 '24 17:03 AbandonedCart

The other issue is that the current build takes about 4 minutes. Building the library adds a full 3 minutes to that. Using a prebuilt library adds 35MB to the repo. Both slow things down quite a bit.

AbandonedCart avatar Mar 29 '24 13:03 AbandonedCart

Ig then dont add it, many people won't like the app size get increased.

Or would be better if you make a fork and put it there, so that those who want to use, can use that fork.

ARIF683 avatar Mar 29 '24 14:03 ARIF683

It's a 35mb increase lol, it's not that deep

FeeninForMore avatar Mar 29 '24 14:03 FeeninForMore

It's a 35mb increase lol, it's not that deep

Need to ask in discord first 😅

ARIF683 avatar Mar 29 '24 14:03 ARIF683

The whole playback process is also slow AF. About a 15 second minimum.

AbandonedCart avatar Mar 29 '24 14:03 AbandonedCart

@AbandonedCart It's really slow, but I think it could be improved, it also doesn't detect subtitles or multi audio, I have a suggestion if you don't want it to take up so much space in the APK, why not, an option to install the player within the same dantotsu something like an addon

moonpic avatar Mar 29 '24 15:03 moonpic

It's not a player, though. It's literally just a virtual server for the torrent. It's passing the updated url back to the existing video wrapper and media object. It plays directly in exo.

AbandonedCart avatar Mar 29 '24 15:03 AbandonedCart

I'm more worried about the build time than the APK size

rebelonion avatar Mar 29 '24 19:03 rebelonion

10 seconds through Actions.

There wasn't much value in making it part of the same project, except for building. Both actions and gradle have a way to download files, so it just downloads a compiled version from a fork of the server that has all the changes.

AbandonedCart avatar Mar 29 '24 19:03 AbandonedCart

I'm open to adding it. Another option is making a torrent flavor and a regular flavor that just redirects to another app

rebelonion avatar Mar 29 '24 21:03 rebelonion

I think kuukiyomi had the right idea by making it a fork of aniyomi. Making it a flavor requires a lot of needless overhead, but including it means it more or less runs whether you use it or not.

I already have a fork that includes it, so I think it might be best to leave it at that. I can focus on improving it and nobody here has to worry about the impact on the rest of the app.

AbandonedCart avatar Mar 30 '24 12:03 AbandonedCart

If torrent support is integrated in the future, it will be crucial to implement corresponding updates to the player to enable embedded subtitle and multi-audio support. Without these necessary adjustments to the player, users would be unable to select subtitles or switch between audio options seamlessly.

ni3x avatar Mar 30 '24 13:03 ni3x

Yet another reason to keep it as a fork.

AbandonedCart avatar Mar 30 '24 14:03 AbandonedCart

but including it means it more or less runs whether you use it or not.

That could easily be a setting.

Lemme put it this way. It's something I'd probably use, and at that point I may just want it in the app. Still unsure tho

rebelonion avatar Mar 30 '24 20:03 rebelonion

If torrent support is integrated in the future, it will be crucial to implement corresponding updates to the player to enable embedded subtitle and multi-audio support. Without these necessary adjustments to the player, users would be unable to select subtitles or switch between audio options seamlessly.

The app already has embedded subtitles

rebelonion avatar Mar 30 '24 20:03 rebelonion

There's a lot of stuff that still has to be worked out in relation to torrents.

most definitely. This is a longer term project. Btw, have you looked apk sizes with splits? I may end up doing that anyways.

splits {
        abi {
            enable true
            reset()
            include 'armeabi', 'armeabi-v7a', 'arm64-v8a', 'x86', 'x86_64'
            universalApk true
        }
    }

rebelonion avatar Mar 31 '24 01:03 rebelonion

I normally build split without universal, since I only use v8a

AbandonedCart avatar Mar 31 '24 01:03 AbandonedCart

If torrent support is integrated in the future, it will be crucial to implement corresponding updates to the player to enable embedded subtitle and multi-audio support. Without these necessary adjustments to the player, users would be unable to select subtitles or switch between audio options seamlessly.

  • [x] Embedded Subtitle
  • [x] Use sub settings
  • [x] Embedded Audio

Where can I test this new changes?

ni3x avatar Apr 01 '24 07:04 ni3x

Ngl you can be difficult to work with, but I doubt he has a vendetta against you. I also don't think even he would argue that he's super skilled yet, but incompetent is a bit strong. For the hardcoded strings, even I've accidentally let some slip through. And I haven't had time to look at the contributor change, but if all he did was put the contributors before the other people, I'd prefer it that way honestly.

rebelonion avatar Apr 08 '24 17:04 rebelonion

Ok dude 👍

rebelonion avatar Apr 08 '24 17:04 rebelonion