pcca-matrix / PCCA-Layout

HyperSpin layout for Attract-Mode
GNU General Public License v3.0
7 stars 6 forks source link

Game count #31

Closed ghost closed 4 years ago

ghost commented 4 years ago

Using the latest commits, I did the following test and observed the following behavior:

This test video: https://youtu.be/a3c9mwyOJpY

This is the console printout of this entire test: console dump.zip

The attract.cfg file for this test: attract.zip

pcca-matrix commented 4 years ago

can't reproduce isn't the same when Track Usage is set to No ? https://github.com/pcca-matrix/PCCA-Layout/issues/25

ghost commented 4 years ago

No, this is unrelated to the latest commits. I tested it again just now. The behavior is still as I described in the first post.

ghost commented 4 years ago

There is a clue - The last system on my list is Daphne. When I delete pcca.stats and then launch AM, pcca generates the pcca.stats file but only the daphne system shows the game count in the main menu page. The rest of the systems don't show anything. If I enter any of these systems the process happens as I described.

My hunch is that the code is accessing some wrong array cell or something like that. Why is the daphne system the only one with the game count? and it does not turn to -1 when I enter its system!

ghost commented 4 years ago

Here is another interesting thing: the pcca.stats file contains 80 systems which is way more systems then I have in my main menu and attract.cfg! This means that the layout is reading stats that are in the attract mode folder structure BUT NOT in his current attract.cfg file! This is an opening for exactly this kind of issue IMO.

It lists all the systems I indeed have in attract mode, but many of them are not in the attract.cfg file I currently use because I did not download HS themes for every system! I have 20 systems listed in the attract.cfg file.

pcca-matrix commented 4 years ago

Ok I understand better , it should work correctly now.

ghost commented 4 years ago

Using the latest commits, I now have abetter view of the issue. I deleted the pcca.stats file and launched AM. It generated a correct list of the systems with their count. I backed up that file without closing AM and then entered three systems: SNK Neo Geo, Nintendo 64 and SNES. Each time when I exited one of those systems back to the main menu it showed a count of -1 BUT if you look at the pcca.stats file, you will see TWO counts - one correct and one -1. The initial count uses the rom list name of the system while the second count uses the display name of the system. The display name is the one that generates the -1. So this tells us it generates a second count when you first enter a system instead of using the initial correct count.

Here are the files for your comparison: pcca stats.zip

pcca-matrix commented 4 years ago

That should do the trick :) (https://github.com/pcca-matrix/PCCA-Layout/commit/b615dc9fef814a2f09770ab48c7a047c562819d9) let me know

ghost commented 4 years ago

If you got another post regarding this issue disregard it - it was a problem in my attract.cfg. Issue resolved! 🥇