Schedule instead of straight download about showroom HOT 19 CLOSED

Lines 1054-1055 are part of a placeholder function (simple_watcher) that doesn't do anything yet. It doesn't get called by main so uncommenting shouldn't change anything, unless you also removed the quotes on lines 1137 and 1144 (in which case explosions are the expected result, not watching).

If you just type python showroom.py "Member Name" it should call the watch function... or it would if I hadn't broken that part with the new index.

from showroom.

Using member name as arguments is failing. It says that the MemberName is not exist. So my work around that works is uncommenting those lines.

I'll put more detail later with screenshots

from showroom.

It's possible that my local code is out of sync with github, but from what I'm seeing uncommenting those lines shouldn't have any effect. The problem lies on line 1130, where main tries to call watch with a member_index variable that no longer exists.

Since I'm going to have to rewrite this anyway, let me make sure I understand what you are trying to do. You want the script to just focus on one room, instead of all the rooms, correct? And instead of going into full watch mode, to read the schedule and wait until they're supposed to come on?

from showroom.

Yes, I want the watching one room is functional and ...
I would also like to be able to schedule on for a set of names, meybe we can add member index number on the index files as an optional index to call, so instead of showroom.py "Member Name" it would able to take showroom.py --indexnum 1,4,5,12

Well, I can make a workaround using a custom index file though

from showroom.

Well you can sort of do this already by creating a custom index ("jdex") file that only has the rooms you want in it, and moving or deleting the other jdex files out of the index directory. But it would be nice if you could just type a list of names on the command line and it would only track those rooms.

from showroom.

Yes, I just added that as an update to my comment. I was confused at first because the current github code tells it watch default_member.json while the code has jdex files.

from showroom.

Ah. Are you on the main branch?

https://github.com/wlerin/showroom should default to new-index, which is the branch currently in development. (And also the one with broken watch--it should still work in the old main branch.)

from showroom.

I am on default branch which is new-index

it still has

MAX_DOWNLOADS = 60
MAX_WATCHES = 50
MAX_PRIORITY = 80
LIVE_RATE = 11.0
SCHEDULE_TICKS = 20
END_HOUR = 0
RESUME_HOUR = 5
DEFAULT_INDEX = 'index/default_members.json'
NEW_INDEX_LOC = 'index

which make me confused a bit

from showroom.

also, the --help shows default-members.json

from showroom.

Ah. Right. I guess I should comment that out and fix the help output too.

Ah. Actually. That index argument does nothing. Hmm.

from showroom.

No rush, I should thank you for this awesome work. Hoping to see the new updates

from showroom.

92c18d6

It probably needs more testing, but that feature should work now.

Also, --all is now the default, if you run it without giving any names. (I suppose I should update the README to reflect this...)

from showroom.

Awesome, going to pull later.

Now I need to think of how to port / integrate it with my Discord bot

from showroom.

Depending on what your Discord bot does... you could always write your own announce.py to send the announce message to the bot instead of to the command line.

I don't think that receives Schedule information though, only when rooms go live.

from showroom.

I want to pass the command from the bot and outputs the result on Discord. I probably better to go with ssh from the bot. I am still learning and sorry for mentioning this here. It's unrelated to the issue

from showroom.

When you get a better idea of what you want to do, you could open a new "issue" and maybe I can add the functionality here (or you can fork it and maybe I'll merge the new features back in, I dunno).

Right now I don't think there's a way to control the script once it starts (either to change things or to get specific information from it), aside from pressing CTRL+C. This is something I've wanted to fix for a long time, but never gotten around to doing it since there hasn't been a pressing need in my use case.

from showroom.

If you want, I can send you my Discord server invite and see how the bot works

and I should close this issue as it is fixed with the recent commit

from showroom.

Yah, that would help. Just PM me on Stage48 or something.

from showroom.

sent on your profile on Stage48

from showroom.