Please Test: Creating Custom Collections
-
@pjft The video was set to private, sorry about that, it should be fixed now, link below. It might help if the video was delayed. Take a look at the video just attached, that is how it worked before.
The memory deal isn't that big of an issue as it is correctable, it will just be hard for all new users who haven't read through the forum posts to know how to correct the issue. You and I spent a ton of time figuring out how to optimize the ComicBook theme to run at VRAM 100 and it did for quit some time, but now with more updates to ES that has changed.
I figured as much with retorpie system. The problem is that it was always controlled by the config file and it was always the last system by default, now it moves around with little user control.
In regards to the system order, would you be fine with, if sorted, [All sorted systems including "Custom Collections"], then "RetroPie", then [Auto Collections - All, Favorites, Last Played] at the end?
Yes that would be fine, but could it also be the case if not sorted? I want to turn sorting off so that I can group my systems by brand - as stated before I like all my Nintendo systems together, all my sega systems, Neo Geo, etc. If it is sorted it goes to alphabetical, so i turn sorting off. I would be more than happy with what you stated above if it worked that way in both sorted and non sorted. Again, i am a single user and these are just my opinions, so by no means is it the correct way.
Thanks again for everything - you do great work.
-
First of all I just want to thank everyone on the forum who contributes time and effort into improving RetroPie, you are all doing amazing work and it is appreciated! Love the ComicBook theme TMNTturtleguy!
I just wanted to mention that I have experienced white screen issues with the Comic Book theme even in EmulationStation v2.4.0RP and currently at v2.5.0 using the 100MB setting. I have always had to run it at 80MB to avoid that white screen issue where some systems don't load images - it looks like EmulationStation runs out of memory/can't cache any more images so it just leaves them out.
This happened for me on an old partially updated image when I tried the ComicBook theme, as well as currently on my fresh install fully updated image. In both cases, 80mb setting works great.
I haven't installed this test build yet - so it seems like the white screen issue isn't caused specifically by it. Could it be a longstanding EmulationStation bug that doesn't appear in all situations?
-
@edge3000 It is very possible you do have issues with ES 2.4 and forward. I am running 2.3.2 at 100 with no issues. There was a period of time when @pjft did amazing work to improve ES performance and also chane the background color from white to black. From the ES where the background color changed to black through 2.3.2 I could run at 100 with no issues. When @pjft first merged the favorites into the main ES beach, not sure what number that is, I could run well at 100 without white screen issues, but did notice reload problems with the rating stars when exiting games. I never updated my main builds to this update. Now with the custom systems build I get white screen glget errors at vram100. So here was a sweet spot in there where verything worked at 100.
This is getting us sidetracked from the main post though, so let's get back to custom systems!
-
@pjft What do you think the timeline will be for getting this into the master branch of EmulationStation?
-
@pjft Now I'm breaking a dogma :)
Why not using
custom-favorites.cfgfor "quick favourite" or "default favourite" and break with using the gamelist.xml files for tagging <favorites>?
You can free the tag for achievments instead :)Are there any advantages in the "first" fav-system (auto-favorits)?
I think with this you can save some code. Please don't get me wrong - it's just a small thinking towards code economizing :)
We still can use the themes ofauto-favorites, of course :) -
@cyperghost I've been thinking about the exact opposite. Why not use tags (something like
<collection>mario</collection>) in the gamelist vs. a separate file. -
@jdrassa If a game was in multiple collections, would it just be comma delimited?
-
@jdrassa @cyperghost both are good ideas. Let me answer in more detail tomorrow on my laptop rather than over the phone.
-
@pjft I am fine with whatever you choose, it was just something I had been thinking about.
@cyperghost specifically speaking about favorites, one argument for keeping it in the gamelist is that there have been discussions of being able to mark the favorites within the normal system gamelist views.
-
Sorry. Lots of work, little time here :( Sorry for the silence and lack of responses.
@jdrassa @cyperghost Summary in a shorter answer than usual, pardon my attempt at brevity.
Rationale for my decision - which doesn't mean it's the best :)
- Favorites as Metadata: I see it as attributes properties of the game. These can even be themable (the heart for favorites, in ES-Kids, for instance, or potentially a trophy for achievements). So, turning "Favorites" into a separate file is not something I'd feel strongly about and would need some convincing. Also, this keeps it compatible with people who used ES-Kids.
- Systems as metadata: one could see it as tags, in that implementation. It could work. Currently, though, the architecture is more that games belong to a system, and not the other way around. We don't have the current system as a metadata element for the game (though we could, I suppose!). I just did it to follow the same structure as "Load Systems" and then "Populate with Games". If the tags were metadata, I'd need to "Populate Database" and then "Organize in Systems". Not impossible, and not that hard, but different to what we've been doing so far, so that's why I kind of defaulted to that approach without really doing a lot of effort in thinking about many options. I suppose, if in the future we move this to SQL or something, we'd have a Systems table with all the systems/collections, and then a SystemGames table with { SystemId, GameId }, and then a Games table. We wouldn't have a Games table with a { Collections } repeated field... I think. :)
@HurricaneFan I need to clean up the code and then submit it. And incorporate some of the most recent feedback which I'm sure I've lost track of in the thread by now. I expect the more people test it, the easier/faster it'll be to be approved, but the code isn't really to my liking at the moment. I'm just pre-empting feedback that would certainly come from others reviewing the code and commenting on it - effectively trying to spare them some time as this will be a big change already.
I haven't had a lot of time, alas, but maybe I can submit it next week, fingers crossed?
We'll see.
Thanks for the support and interest. Is it working well for you and for those testing? That's really all that matters.
-
@pjft I've been testing it more and more as themes are adding the collection themes. I love the create collection based on theme folder. I've added about 4 of those to my system so far and no issues to report.
-
Without reading the 145 posts... has this been merged yet? I'm eager to go through and build an achievement custom collection. Also.. did the post @meleu started about adding gamelist.xml support for achievements get deleted? I can't find it.
-
@hansolo77 not merged yet. Hopefully I'll submit it next week, just haven't had the time to clean up the code.
-
That's cool. I was just wondering. I've been trying to figure out all these other things on my other build, I was just looking for something to distract me on my WORKING system. Although, I'd hate to go and try to add something else new to it and end up breaking it lol.
-
@pjft How's the code clean up coming along with this feature?
-
@hurricanefan I apologize, I really haven't been having the time to move this forward, much to my chagrin.
The "good" news is that I do have a hard deadline to finish this off before end of month, as then I'll be moving countries, and I won't have a proper laptop for a while. So I do need to finish this off soon. I'll keep you posted here, sorry.
-
@pjft Real world stuff takes priority. Good luck in your move! I just moved states this summer, I know how stressful moves can be.
-
@hurricanefan Thanks.
Yeah, a bit hectic as I try to close everything off in my current work, while sorting out the move with the kids and all, and then trying to ensure a smooth, running landing for all when we arrive.
So yeah. But I really wanted to close this off soon, otherwise even I will forget what I did there :)
-
@pjft, seconding what @HurricaneFan said above, my best wishes for your move as well.
-
Thanks all.
I added the final tweaks to the code and just submitted the PR:
https://github.com/RetroPie/EmulationStation/pull/210
In particular, @TMNTturtlguy I finally changed the sorting option so that it will sort all systems by name, and then at the end we'll have "RetroPie", then the automatic collections ("Collections", "All", "Favorites" and "Last Played"). I also added the "Exit editing mode" to the collections menu, just in case.
Thank you all for the testing, usage, feedback and support. I hope this will be useful for everyone.
For everyone who has been using it on an ongoing basis and testing it (@dmmarti , @HurricaneFan, @alphatoanant , @hansolo77, @cyperghost and others who aren't tagged here yet), if you'd want to drop by the PR and share your experience (how long have you used it for, and whether it has any pending issues or if it's good to go from your testing) that may help it be approved faster, or at least alleviate concerns with the change, as it is a big change.
Thanks.
@alphatoanant Thanks for the wishes as well.
Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.
Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.