this post was submitted on 31 Oct 2023
5 points (100.0% liked)

Toronto Maple Leafs

313 readers
1 users here now

Be polite. A reminder: Fan's is obviously a short form for 'Fanatics' so disagree politely. No flaming and/or Racism.***

founded 3 years ago
MODERATORS
 
Match up Time
LAK LAK - TOR TOR 08:00PM EDT

 

I am open source! Report issues, contribute, and fund me on my GitHub page!

all 8 comments
sorted by: hot top controversial new old
[–] [email protected] 3 points 1 year ago

The bot is Alive!

[–] [email protected] 2 points 1 year ago* (last edited 1 year ago) (1 children)

Uh, I think there's a bug in the bot. It's not supposed to make this daily thread if you don't have any game types in COMMENT_POST_TYPES. I'll have to look into this over the weekend. I haven't really been able to find the time to work on the bot this week.

Edit: I created an issue for this here: https://github.com/dandroid126/lemmy-nhl-gdt-bot/issues/36

[–] [email protected] 1 points 1 year ago (3 children)

I don't think I'm running the latest image, haven't had a chance to update it and pull the latest, I was going to make sure I was on the latest before I bring it up

[–] [email protected] 2 points 1 year ago (1 children)

So I actually gave this some more thought. I think having a daily thread even when all the games are full posts rather than comments would be nice in the case of [email protected], so during the playoffs you can see an overview of all of the games. However, it’s still useless in your case. I think the way I’m going to do it is if there is 2 or more games in a day, the daily thread gets made. I am testing the change today and hopefully will push it out before the Leafs game. I think your most recent daily thread will get stuck as featured, so you might need to unfeature it manually.

[–] [email protected] 2 points 1 year ago (1 children)

That seems like a great idea. It will still have niche uses for a single team that way. I.e. split squad preseason game days

[–] [email protected] 1 points 1 year ago

That change is now live!

[–] [email protected] 2 points 1 year ago

I haven't made an update to that area in the lastest image.

I heavily rewrote the logic last weekend to optimize what was being downloaded from the NHL API, and I broke a few things and just haven't gotten around to fixing it.

Before I was downloading all the data for every game every 30 seconds, then discarding the games that weren't relevant right now. But I updated it so it doesn't download anything not from your selected teams and for games that are more than an hour away from starting. This was a much needed optimization, but it introduced several bugs, including this one.

I need to write more unit tests to catch these things! 😭