Versatile C++ game scraper: Skyscraper

AnalogHero

@muldjord Then its fine. Im also using .uae files with HoraceandSpiders whdload autobooter.

SammyBoy

This post is deleted!

dudleydes

@muldjord I am getting a build error when trying to compile on Linux Mint 17 Qiana Cinnamon 64-bit. Running dpkg-query -l qt5* confirms that I have qt5-default installed (version 5.2.1+dfsg-1). The error occurs when compiling src/fxshadow.cpp. Full output when running make with error message can be found at https://pastebin.com/b8805LVr.

mitu

@dudleydes Seems that the error comes from a function introduced in a newer version of Qt5 (5.3) - http://doc.qt.io/qt-5/qcolor.html#qPremultiply.

dudleydes

@mitu Thanks for the reply. Mint 17 appears to only have version 5.2.1 available in the Software Manager so I'll look to upgrade to Mint 18.

SammyBoy

@muldjord you should consider asking @herb_fargus to allow Skyscraper to be installed from inside the RetroPie menu

muldjord

@dudleydes Yes, Skyscraper requires Qt5.3. I'll add a note about that in the instructions.

@SammyBoy I would love to have Skyscraper in the setup script. I think with the new "simple mode" it would make sense. So it should just compile from source and run it in simple mode. I will look into it at some point.

SammyBoy

@muldjord Did you read my message on Reddit?

bleuge

Is there any cause why i am getting some dupes?
For example, \fba\galaga.zip, i got two exact entries in the gamelist.xml file for fba.

Also, some downloaded pngs are not valid (for example 720.png for arcade set). This could be that the downloaded png is bad (maybe skyscraper could check this), or that skyscraper saved it wrong?

Thanks everyone!

muldjord

@bleuge Concerning doubles, please create pastebin's (https://pastebin.com/) for the following and give me the links:

Paste the output from "ls -l /home/pi/RetroPie/roms/fba"
Paste the output from "cat /home/pi/RetroPie/roms/fba/gamelist.xml" (if that's where you have it)

Which version of Skyscraper are you running? I actually do check the images which is why I find it odd that you get the png errors. Is it only for 'screenscraper' you get those png errors?

bleuge

@muldjord

i connected to windows host, these are the pastebins

https://pastebin.com/fDr4CkXM

https://pastebin.com/rWXYWWnp

I use last skyscraper cloned from repo (git clone ...)

I've made a non-exhaustive check of png images, it seems only bad images in arcade folder.

Also, regarding the hack-link bug. When you fixed it, i re-scrapped everything with --updatedb option, to (as far i understood) update everything. But i'm finding still many wrong images of hack-link associated to games.
I guess this could not be fixed as they are in my localdb, and filename is not related to hack-link so skyscraper is getting these files again.

So, the solution is clear (delete) all my localdb folders and rescrape everything...?

Thanks muldjord!

muldjord

@bleuge Well, the first problem with the doubles is pretty obvious. You have the galaga.zip file in both the fba folder and the fba/samples folder. Skyscraper also scrapes subdirs. If you don't want this, please us the '--nosubdirs' option.

If you run all scraping modules with the '--updatedb' option, the hack-link entries in your localdb's should go away without the need for deleting the entire dbs/platform folder. If it doesn't maybe it's because you're skipping existing entries?

I'll check the arcadedb module to see if anything is wrong with the image check.

EDIT: Can you provide a filename for a game that returns a faulty png?

bleuge

@muldjord I've seen faulty pngs in 720 (arcade) and alien3 for example. This is not something important i think, i could update this files by hand or whatever.
The files don't start with the typical PNG header (PNG magic in the first bytes), but they end with a IEND tag (marking end of file in a png).
As i said, it's not important. But maybe if it's easy and don't slow skyscraper, checking that is a png file prior to saving it could be nice.

This is part of the script i used to rescrape the sets:

Skyscraper --updatedb -t 8 -p coleco --videos --unattend --skipped -s openretro --pretend
Skyscraper --updatedb -t 8 -p coleco --videos --unattend --skipped -s thegamesdb --pretend
Skyscraper --updatedb -t 8 -p coleco --videos --unattend --skipped -s screenscraper --pretend
Skyscraper --updatedb -t 8 -p coleco --videos --unattend --skipped -s localdb

I used --skiped because i wanted that even don't recognized files will be included in the gamelist (so i could check visually for not scraped files)

muldjord

@bleuge Did you delete the non-screenscraper localdb's? Otherwise there's no reason to use '--updatedb' on those, it just wastes bandwidth for the sources since your cached data is completely the same.

Concerning the png's I have no idea why it doesn't have a header. I DO check the images and they are saved using the Qt image function which creates the header itself. So I'll need to think that over for a bit to figure out why it would ever do that.

muldjord

2.3.1 is coming along nicely. I've implemented the scummvm.ini parsing which is really cool, so thank you for that suggestion @stoo .

I've also implemented numeral checking on titles. Which basically means that if a file is called "Blah 4" and the returned title is "Blargh" it won't even check them, it just skips it. The default numeral is "1" so "Blah 1" and "Blah" is a match.

muldjord

Skyscraper 2.3.1 released: https://github.com/muldjord/skyscraper

Fixed 'players' tag to always conform to a 1-digit format
Now filters out ".hack-Link" results from 'screenscraper' to avoid bad localdb data
Added note to output about how many new resources have been added during scraping run
Added 'color="#fffff"' option to stroke effect for the geeky people (including me of course)
Conformed 'game tags' to 'Platform, Action' format
Fixed so 'localdb' folder isn't created inside dbs media folders
Optimized the mameMap a bit
Improved the searchMatch system to also consider numerals
Now looks up 'scummvm' dummy files in 'scummvm.ini' and uses the correct game name

This release contains some user requests, the screenscraper 'hack-Link' fix and a bunch of optimizations. Most prominently Skyscraper is now fully aware of game numerals ("Game 4" or "Game IV") and acts upon them when comparing results. This should mark the end of game sequels being matched with results that don't have the same numeral in the title as the filename. A quick note: You might notice that you have fewer "game found" with this release. This is intentional. I've changed the default minimum match percentage to 65 (from 50 before) to eliminate more false-positives. And combined with the more strict numeral checking, that will result in less false-positives, which might look like it finds less correct results. That should not be the case. The quality of the results are just more precise.

Let me know what you think and happy scraping!

bleuge

Thanks very much!!! Skyscraper is getting better and better!

jwcbronski

Skyscraper is great. But I still get the ".hack-Link" results from screenscraper with version 2.3.1.

muldjord

@jwcbronski Yes, you need to rerun it with '--updatedb'. Otherwise it uses the cached results (which still contain the hack-link entries). Running it with '--updatedb' will refetch the data from screenscraper and overwrite the faulty hack-link results effectively removing them.

jwcbronski

@muldjord I deleted the whole .skyscraper folder before I installed and ran Skyscraper 2.3.1. So there where no cached results. I started from scratch and got the ".hack-Link" results again. That was just 2 hours ago.