#54 hearthbeat (keepalive ?)

Merged
LecygneNoir merged 4 commits from Zykino/prismedia:hearthbeat into develop 4 months ago
Zykino commented 4 months ago

Use a bit of Youtube’s quota to tell them not to invalidate the apikey.

Use a bit of Youtube’s quota to tell them not to invalidate the apikey.
Zykino added 2 commits 4 months ago
a725e848ab Add an option to use some credits easiely
Zykino force-pushed hearthbeat from 7a691a1fae to 29b1747c3e 4 months ago
Zykino changed title from WIP: hearthbeat to hearthbeat (keepalive ?) 4 months ago
Poster
Owner

The idea is interesting, indeed if you get caught by google for no use and they reduce your quota to 0, regaining some quota is a real PITA, this may avoid this 👌

The idea is interesting, indeed if you get caught by google for no use and they reduce your quota to 0, regaining some quota is a real PITA, this may avoid this 👌
LecygneNoir requested changes 4 months ago
prismedia/upload.py
Usage:
prismedia --file=<FILE> [options]
prismedia -f <FILE> --tags=STRING [options]
prismedia --hearthbeat
LecygneNoir commented 4 months ago

--file is mandatory, shouldn't we set it optional to allow launching prismedia with --hearbeat only?

So --file becomes optionnal, perhaps exclusive with --heartbeat. With a check so if there is not --file nor --hertbeat present, we disply the help? (One of two is mandatory)

`--file` is mandatory, shouldn't we set it optional to allow launching prismedia with `--hearbeat` only? So `--file` becomes optionnal, perhaps exclusive with `--heartbeat`. With a check so if there is not `--file` nor `--hertbeat` present, we disply the help? (One of two is mandatory)
Zykino commented 4 months ago

Yes one of the two is mandatory and both are exclusives (at least in my mind).

I had to add this usage otherwise I had an error. But I added quickly, there may be a better syntax to highlight what we are both thinking.

Yes one of the two is mandatory and both are exclusives (at least in my mind). I had to add this usage otherwise I had an error. But I added quickly, there may be a better syntax to highlight what we are both thinking.
LecygneNoir marked this conversation as resolved
prismedia/upload.py
Only relevant if --playlist is set.
--progress=STRING Set the progress bar view, one of percentage, bigFile (MB), accurate (KB).
--hearthbeat Use some credits to show some activity for you apikey so the platform know it is used and would not put your quota to 0 (only Youtube currently)
LecygneNoir commented 4 months ago

I think we may add a section for heartbeat in README as documentation, to explain the way to use it.

Do you plan to use cron? Or some daemon? Perhaps a beehive example? ;-)

Thanks!

I think we may add a section for heartbeat in README as documentation, to explain the way to use it. Do you plan to use cron? Or some daemon? Perhaps a beehive example? ;-) Thanks!
Zykino commented 4 months ago

Yeah I forgot to add the readme and explain "just use cron on Linux or add it in the Windows sheduler to be triggered once a month". Not sure if more details are needed.

Yeah I forgot to add the readme and explain "just use cron on Linux or add it in the Windows sheduler to be triggered once a month". Not sure if more details are needed.
Zykino marked this conversation as resolved
LecygneNoir reviewed 4 months ago
prismedia/yt_upload.py
if playlist["snippet"]['title'] == playlist_name:
return playlist['id']
pageToken = ""
while pageToken != None:
LecygneNoir commented 4 months ago

Oh you find a way for youtube pagination, great! 👏 💪

Oh you find a way for youtube pagination, great! 👏 💪
Zykino marked this conversation as resolved
LecygneNoir reviewed 4 months ago
prismedia/yt_upload.py
LecygneNoir commented 4 months ago

Little typo: no space before the : in english :-)
(BTW, I'll probably use the release for this MR to track the last spaces that remains in code 😂 )

Little typo: no space before the : in english :-) (BTW, I'll probably use the release for this MR to track the last spaces that remains in code :joy: )
Zykino commented 4 months ago

I think I fixed everything. But if there are more/other things, feel free to fix the rest :)

I think I fixed everything. But if there are more/other things, feel free to fix the rest :)
LecygneNoir marked this conversation as resolved
Zykino added 1 commit 4 months ago
LecygneNoir merged commit ca733e0dc3 into develop 4 months ago
Zykino deleted branch hearthbeat 4 months ago

Reviewers

LecygneNoir requested changes 4 months ago
The pull request has been merged as ca733e0dc3.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.