Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Retrogaming.cloud grid provider #470

Closed
wants to merge 2 commits into from
Closed

Added Retrogaming.cloud grid provider #470

wants to merge 2 commits into from

Conversation

wranders
Copy link

@wranders wranders commented Jul 20, 2017

This aims to replace ConsoleGrid with Retrogaming.cloud as the grid image provider.
Has worked as intended on every ROM I've tested so far.

Two items to note, could be considered drawbacks associated with RG.C:

  • Games have to be named exactly as the appear on the site. RG.C allows special characters that are not allowed in file paths, so a ROM name omitting a colon as a separator will fail to match (Eg.). This may be worked around by not using a literal search, but that seems to bring up many unrelated results.
  • RG.C has an unspecified request rate limit. It appears to be around 150 per minute, but since 2 requests are required to retrieve the image URL, that number is half. Testing shows 70-80 ROM queries before the limit is triggered for an unspecified cool-down.

(Repost of #467 since HEAD changed.)

wranders added 2 commits June 15, 2017 14:57
- Modified: Game query
- Modified: JSON location of grid image URL
@wranders wranders changed the title Retrogaming cloud provider Added Retrogaming.cloud grid provider Jul 20, 2017
@technobone
Copy link

Please can you tell me where or how do I get a compiled version of ice with this pull request. Thanks

@wranders
Copy link
Author

@technobone I don't have a compiled version. Only way at the moment is to clone and run from the shell. If you clone my latest branch, it includes this PR and the other that uses the new shortcut format.

@technobone
Copy link

Ok thankyou I'll take a look when I get chance. Thanks agin

@wranders wranders closed this by deleting the head repository Apr 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants