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

[Meta] Suggest issues deserving bounties #2199

Open
TheFrenchGhosty opened this issue Jun 24, 2021 · 16 comments
Open

[Meta] Suggest issues deserving bounties #2199

TheFrenchGhosty opened this issue Jun 24, 2021 · 16 comments

Comments

@TheFrenchGhosty
Copy link
Member

This is a meta issue where organizations members, and users in general can suggest issues that deserve to have bounties added to them.

@syeopite
Copy link
Member

syeopite commented Jun 24, 2021

And maybe we should also bump the existing bounties since no one is tackling them.

@SamantazFox
Copy link
Member

SamantazFox commented Jun 25, 2021

What @syeopite said, plus #2098, #1762 and #1561, maybe?

@TheFrenchGhosty
Copy link
Member Author

@syeopite

Any major features of Invidious that are broken should also have one. Like #1979 or the whole lsquic fiasco.

Bounty added on #1979. Thank you :)

About lsquic I really don't know what has to be done, so I will let you guys handle it to be sure it's "complete".

Any major features that needs or should be added and would be difficult to implement should also have one. Add Livechat on Livestreams #438, Circumventing geo-restrictions #92, How does decentralization / federation work and how to improve it? #879 etc.

Waiting for @Perflyst

Since we started adding bounties Perflyst wanted us to only add them on bugs (and not enhancements/feature requests). Back then it was mainly because there was so much problems with Invidious, but now I guess it's time to add some bounties to enhancements/feature requests.

@TheFrenchGhosty
Copy link
Member Author

@SamantazFox

#2098

We have to discuss if we really want Google SSO on Invidious, in my opinion it's a pretty bad thing to have on a service mainly made to avoid Google.

#1762

If it's only an issue when decrypt_polling is enabled... is it really a problem? According to the description you added on your config.example.yml rewrite it allows access to "secured videos", I really don't understand what this mean, can you please explain?

#1561

I know what PubSub is however I'm wondering... is it really useful to have? I think we had this discussion before and if I remember correctly the answer was yes, but I would just like to be sure.

@Perflyst
Copy link
Contributor

or the whole lsquic fiasco

If someone can define what needs to be done I guess we can add a bounty. I can't estimate how much work this will be.

Add Livechat on Livestreams #438

Even though it is my own feature request I still think this feature is not very important compared to other bugs / improvements left
We can put some bounties on most wanted feature requests I guess

Circumventing geo-restrictions #92

Sounds good to me, go for it

How does decentralization / federation work and how to improve it? #879

This looks more like a discussion to me than some feature request

@SamantazFox SamantazFox added this to Work In progress in Todo - Invidious Jun 25, 2021
@SamantazFox SamantazFox moved this from Work In progress to Ongoing discussions in Todo - Invidious Jun 25, 2021
@SamantazFox
Copy link
Member

We have to discuss if we really want Google SSO on Invidious, in my opinion it's a pretty bad thing to have on a service mainly made to avoid Google.

Some people want to use their G-account to continue to interact with the content creators/community (comments/likes) without being too much tracked and annoyed by Youtube itself. And if we work better on accessibly, it could be another reason (than privacy) why people choose us.

If it's only an issue when decrypt_polling is enabled... is it really a problem? According to the description you added on your config.example.yml rewrite it allows access to "secured videos", I really don't understand what this mean, can you please explain?

Some videos, mostly music, contain a form of DRM that requires a small algorithm to run on the client side to "verify" that they can play the video. decrypt_polling is a "job" (a function ran periodically by crystal) that extracts this algorithm. Because this function is currently broken, issues like #1947 happen.

I know what PubSub is however I'm wondering... is it really useful to have? I think we had this discussion before and if I remember correctly the answer was yes, but I would just like to be sure.

I honnestly have no clue about what PubSub is, lol. But from what I remember from "hovering the code", it's required for subscriptions or something?

@unixfox
Copy link
Member

unixfox commented Jun 26, 2021

I know what PubSub is however I'm wondering... is it really useful to have? I think we had this discussion before and if I remember correctly the answer was yes, but I would just like to be sure.

Yes it is very important, for big instances it allows to instantly get notified once a new video comes out instead of waiting 5 days that Invidious refresh all the 100 000 channels in the database.

@jtagcat
Copy link

jtagcat commented Jun 28, 2021

If the budget is not all booked, allocating it to items in to-dos might be of interest. Though, where is the limit — 2$ bounties for minor bugs, paper cuts, feels wrong.

@lunainvictum

This comment has been minimized.

@SamantazFox

This comment has been minimized.

@SapeurPonpier
Copy link

I think #2236 deserves a bounty. Using invidious on iOS has become tedious.

@TheFrenchGhosty
Copy link
Member Author

@SapeurPonpier Thank you for the suggestion, bounty added!

@SamantazFox SamantazFox unpinned this issue Nov 15, 2021
@github-actions
Copy link

This issue has been automatically marked as stale and will be closed in 30 days because it has not had recent activity and is much likely outdated. If you think this issue is still relevant and applicable, you just have to post a comment and it will be unmarked.

@github-actions github-actions bot added the stale label Sep 29, 2022
@unixfox unixfox removed the stale label Sep 29, 2022
@ChunkyProgrammer
Copy link
Contributor

I think the automatic db migration could be a good candidate for a bounty. It's blocking a few PRs #2254 (comment)

@syeopite
Copy link
Member

I think we should consider putting up bounties for whoever can discover the source(s) of the memory leaks in Invidious

@TheFrenchGhosty
Copy link
Member Author

@syeopite

I think we should consider putting up bounties for whoever can discover the source(s) of the memory leaks in Invidious

Sure, we should discuss it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Ongoing discussions
Todo - Invidious
Ongoing discussions
Development

No branches or pull requests

9 participants