Skyscraper now officially part of RetroPie, please test

muldjord

@Silent I'm not really sure what you mean by "wildly different output". The gui uses a few options to make it better suited for use in a GUI. Other than those options (which basically just skips the initial questions about overwriting and skipping) the output is similar to running it on command line.

Silent

@muldjord said in Skyscraper now officially part of RetroPie, please test:

I'm not really sure what you mean by "wildly different output"

Paths, for once. You can get them from reading the source code or from pausing the scraper at a right moment (at "COMMANDLINE USED:" line), but I feel like it's still not foolproof enough.

Don't get me wrong, I am perfectly able to use it from CLI as is, but I am trying to think of as user friendly option as possible.

muldjord

@Silent So basically you want the default command line options used by the GUI to be configurable? I'm not against that although some of them are locked. RetroPie doesn't allow questions during running a script for instance, so --unattend is locked. It could make sense though. Some of them would be locked, but it could have some "extras" that you could provide yourself. But I'm also thinking that you already have the config.ini for that purpose. Which is actually better.

spurlingD

@muldjord i have installed it with retropie but when i type in skyscraper it not do anything

muldjord

@spurlingD If you typed in "skyscraper" that's why. The command is "Skyscraper".

Silent

@muldjord said in Skyscraper now officially part of RetroPie, please test:

@Silent So basically you want the default command line options used by the GUI to be configurable?

Kinda. Cases where it'd be useful would be any "odd" stuff you'd want to do, like:

scrape single rom (afaik you cannot do that with config.ini now)
scrape a range of roms (you can use startat and endat in config.ini but it's awkward to edit every time)
scrape with a query (probably impossible from config.ini?)
easy access to any upcoming features whcih didn't make it to GUI yet (like now - vacuuming, ES scraper)

muldjord

@Silent For your examples, why not just use the command line? I don't see how a free text field would provide a user friendly way of running commands, quite the contrary. You don't have tab completion in the GUI, so you'd have to know the exact filename to do what you want (and not do any spelling errors). And for that you'd need to look them up on the command line beforehand. In which case you could just run the command there. If the user knows about these features you describe, they have already checked the output of --help where the commands are listed. So they even know how to run the executable from the correct location. So yeah, I don't think this makes much sense. I'd much rather put in some better help text telling people how to actually run it from command line. I mean, you've already exited EmulationStation to even open up the GUI. So the command line is right there.

spurlingD

@muldjord i have typed Skyscraper and it say command not found

mitu

@spurlingD If you installed Skyscraper using the RetroPie scriptmodule, then the command is located in /opt/retropie/supplementary/skyscraper/Skyscraper. If you add the installation folder to your $PATH, then you don't have to type the full path to the command.

Silent

@muldjord said in Skyscraper now officially part of RetroPie, please test:

For your examples, why not just use the command line?

Figured I'd post the way I "solved" my personal gripe I had with this approach:

I never use "Use ROM folders" so I could just add those to config.ini as defaults:

inputFolder="/home/pi/RetroPie/roms"
gamelistFolder="/home/pi/.emulationstation/gamelists"
mediaFolder="/home/pi/.emulationstation/downloaded_media"

and now I don't need to provide those as -o and -g arguments, like RetroPie-Setup script does.

Tested it when scraping for a single ROM and it worked flawlessly!

EDIT:
In setup like this, I hope that commandline parameters take precedence over what is in INI file? Just in case RetroPie-Setup ever provides paths different to these.

EDIT2:
Does --pretend imply --refresh? When scraping for single ROMs it's always redownloading from the website, bypassing the cache.

spurlingD

@mitu how do i get it show it not in text from but the menu

mitu

@spurlingD I don't understand what you mean. Can you give an example ?

spurlingD

@mitu like this 1539341719679-2d4d68da-bea6-4f9c-a44f-bee87d3d5ecd-image.png

mitu

@spurlingD Have you read the docs ? It's explained in there.

brimby

I'm trying to scrape igdb, but it keeps rejecting my api key. I'm pasting the key directly in (not trying to type it out). Here is my command line entry:

/opt/retropie/supplementary/skyscraper/Skyscraper --unattend --skipped -g /home/pi/.emulationstation/gamelists/n64 -o /home/pi/.emulationstation/downloaded_media/n64 -p n64 -s igdb -u MYPASTEDAPIKEY --videos --nobrackets

Does anything look wrong with that line? The error message is: "The key you provided with either the '-u [key]' command line option or the 'userCreds="[key]" variable to config.ini does not seem to work. Now quitting..."

I'm on version 2.9.5

Update: I just tried generating a new key, but that one didn't work either.

fuzznuzzbit

Would it be easy (or is there a way) to make a configuration to move the location of the cache? I run everything off of USB - roms, gamelist, etcetera, and would love have the Skyscraper cache and db in a folder on the usb as well.

Edit: Nevermind, it was as easy as editing the .ini file! Really like this application!

muldjord

@fuzznuzzbit Glad you found the solution and like the software :)

And happy new year everyone!

muldjord

@mitu In preparation for the 3.0.0 release of Skyscraper, it would be nice if any scraping ended with a dialog that said "All resources cached. Once you've gathered all the data you need, please remember to scrape again with the localdb module which will generate the game list" or similar. And obviously, when scraping with localdb it should say "Scraping complete, game list has been generated" or similar.

mitu

@muldjord when do you intend to release it ? The Wiki would also need to be updated.

muldjord

@mitu Hard to give an estimate as I am currently battling IGDB's unstable servers to integrate their new API. And I also have some stuff with cache entry editing I'd like to get in. So... Maybe 2 weeks? Rough estimate.