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

New game definitions created on Steam Deck not being recreated on PC installation #80

Open
dc-grimm opened this issue Mar 9, 2023 · 13 comments
Labels
bug Something isn't working

Comments

@dc-grimm
Copy link

dc-grimm commented Mar 9, 2023

Hi David,

I created a definition for syncing god of war ps2 and some other emulated games on the Steam Deck.

When Syncing on pc these don't get created.

@DavidDeSimone
Copy link
Owner

This is an area I am trying to improve in 0.18 - try opening the app on windows, waiting a few seconds, and then restarting the app on windows.

The issue if that while we do sync game definitions across the platforms, we don't refresh the UI when that operation is complete, which can lead to user confusion.

it is also possible there is a bug in the sync logic, but I'd like to start with that step first.

@dc-grimm
Copy link
Author

dc-grimm commented Mar 9, 2023

ps. I tried downloading the user_overrides.json from the cloud and replaced it in %appdata%\OpenCloudSave but that seems to be the default game list without updates.

@DavidDeSimone
Copy link
Owner

ps. I tried downloading the user_overrides.json from the cloud and replaced it in %appdata%\OpenCloudSave but that seems to be the default game list without updates.

Hmmmm, so that means that your user_overrides didn't update to the cloud. I'll investigate this more. For now, you may need to recreate the definition across your devices - sorry about that!

@DavidDeSimone DavidDeSimone added the bug Something isn't working label Mar 9, 2023
@dc-grimm
Copy link
Author

dc-grimm commented Mar 9, 2023

Update: After a few syncs on the Deck and manual adding it to the PC one its now showing the new entries.

It seems that the sync on the pc takes way longer than on the deck though

@dc-grimm
Copy link
Author

dc-grimm commented Mar 9, 2023

Okay. It was still busy simulating so I killed it.
I think I broke it
image

@dc-grimm
Copy link
Author

dc-grimm commented Mar 9, 2023

Seems like its listing everything from windows, users/username, program files +x86, and other folders.

I suspect its because I have an empty path in one of my definitions.
I couldn't manage to remove it with the interface so I deleted it from the json.

Will retest.

@DavidDeSimone
Copy link
Owner

yeah good catch - it may have been defaulting to a "root" path so it was syncing everything under the sun. That is a bad footgun, and I'll add some more protections around that.

Sorry you are having these issues! I'll work to try and correct some of these rough edges for next release.

@dc-grimm
Copy link
Author

dc-grimm commented Mar 9, 2023

Na man. No problem.
Its all good. Having this tool is already a win, and helping to make it better makes it even better.
Unfortunately I don't have the skills you are employing here or I would have loved to help more.

Btw. I had trouble installing it on pc. It just popped up a cmd window and closed without options.

Not sure how it finally got installed to be honest. I opened the installer with zip and ran the application from extracted and after that I found it installed under program files. So that might be something else to also look at in future.

@DavidDeSimone
Copy link
Owner

For some guardrails on large syncs, we will now have this:

Screenshot 2023-03-09 at 8 07 16 PM

Screenshot 2023-03-09 at 8 07 08 PM

I've also put a cap on log files to prevent them from blowing up to a huge size.

@hicwic
Copy link

hicwic commented Mar 23, 2023

the file user_overrides.json doesn't seems to sync for me too. The one on my drive appear to be the original one.
for now i'll try to recreate definition on each device

@DavidDeSimone
Copy link
Owner

This slid to 0.19, but is still on my RADAR

@DavidDeSimone
Copy link
Owner

So I figured out the root cause - even though I was syncing user_overrides.json, I was writing to it right before the cloud sync operation. So user_override would always have a newer timestamp compared to what was on the cloud, and it would not sync (and often get in weird states).

@gingerr
Copy link

gingerr commented Dec 10, 2023

Stumbled upon the same problem when trying out OpenCloudSaves to sync Zelda BotW between Steam Deck and PC. I will attempt next to sync the user_overrides.json manually once i localized it on the steam deck.

Do you plan to build and distribute a new OpenCloudSaves version which includes #104 ?

Thank you for OpenCloudSaves!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants