search

Badaro / MTGODecklistCache

Cache in JSON format of tournaments posted on MTGO, Manatraders, Melee and Topdeck Websites
34 stars 4 forks source link

Updater broken #48

Closed alex-mckinley closed 4 months ago

alex-mckinley commented 4 months ago

I noticed that this hasn't been updated in a few days. I am not sure how automated it is, so I started running the updater app myself. When run, the updater app gives this error: 

NullReferenceException: Object reference not set to an instance of an object.
   at MTGODecklistCache.Updater.MtgMelee.Client.MtgMeleeClient.GetPlayers(Uri uri, Nullable`1 maxPlayers) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.MtgMelee.Client/MtgMeleeClient.cs:line 81
   at MTGODecklistCache.Updater.MtgMelee.Analyzer.MtgMeleeAnalyzer.GetScraperTournaments(MtgMeleeTournamentInfo tournament) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.MtgMelee.Analyzer/MtgMeleeAnalyzer.cs:line 24
   at MTGODecklistCache.Updater.MtgMelee.TournamentList.GetTournaments(DateTime startDate, Nullable`1 endDate) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.MtgMelee/TournamentList.cs:line 32
   at MTGODecklistCache.Updater.MtgMelee.MtgMeleeSource.GetTournaments(DateTime startDate, Nullable`1 endDate) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.MtgMelee/MtgMeleeSource .cs:line 22
   at MTGODecklistCache.Updater.App.Program.UpdateFolder[T](String cacheRootFolder, ITournamentSource`1 source, DateTime startDate, Nullable`1 endDate) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.App/Program.cs:line 53
   at MTGODecklistCache.Updater.App.Program.Main(String[] args) in /mnt/c/Users/amcki/Documents/decklist_feed/MTGODecklistCache.Tools/MTGODecklistCache.Updater.App/Program.cs:line 44

I wonder if this broke the automation of the update of this repo

Badaro commented 4 months ago

It's fully automated running once a day, but there may be something breaking. I'll check here and make a fix.

Badaro commented 4 months ago

@alex-mckinley Yes, it's breaking the automation, since this causes an error the process stops before pushing+commiting. I've manually pushed the mtgo tournaments while I investigate this issue.

Badaro commented 4 months ago

Temporarily disabling to resume mtgo scraping while I investigate the issue. Discussion for that will be done on #49.

Badaro commented 4 months ago

Moving the melee discussion to a more proper location: https://github.com/Badaro/MTGODecklistCache.Tools/issues/11