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

Remove Supermodel.ini from future releases #6

Open
kathyrollo opened this issue Jul 20, 2022 · 2 comments
Open

Remove Supermodel.ini from future releases #6

kathyrollo opened this issue Jul 20, 2022 · 2 comments

Comments

@kathyrollo
Copy link

Hi @trzy , opening this ticket as requested in the forums.

Context:
During update, users tend to download the latest ZIP and simply overwrite all their old files with the new ones. Controller config is often cited as the part that needs work and having a Supermodel.ini included in the ZIP overwrites existing custom controls if users forget to back it up, which they will haha. Not a heavy task to back up, but an added cognitive load which users have to remember when doing so.

Proposed Solution:

  • For new users/fresh installs, the missing INI is not a problem because they are already instructed from the Help documentation to generate one if it is missing. Nothing changes from this POV.
  • For existing users with configs, the accidental overwrite is a real possibility. Removing the INI from future releases should require no additional code and Supermodel effectively behaves the same. All files are safely overridden on update, with the safety of avoiding a possibly frustrating mishap. This is the quick no-code solution.

Possible Approach: (good to have, not mandatory)

  • -config-inputs generates the INI if it does not exist. This is current behaviour, so no changes here.
  • -config-inputs detects an existing INI and asks the following in the cmd prompt:
Supermodel.ini exists. Do you wish to overwrite and reset inputs to default (y/n)? y
Creating new INI file...
(proceeds to the usual input config)

Supermodel.ini exists. Do you wish to overwrite and reset inputs to default (y/n)? n
Modifying existing INI...
(proceeds to the usual input config)

y = creates a new INI, all custom mappings reset to default
n = uses existing INI and appends to it, previous custom mappings preserved

The command is very well doing this now under the hood, but perhaps a couple of decision/confirmation messages further improves user experience when they are already daunted by the cmdline interface. This is also the feedback I hear from the emulation group I'm in, so I try to make the YouTube tutorials to make Supermodel config easier for them.

Thanks for reading Bart. Best of luck for your crunch time! :)

@trzy
Copy link
Owner

trzy commented Aug 13, 2022

Thanks for this! I just saw it. Not sure when I'll get to it but will have a look at some point.

@BenoitAdam94
Copy link

BenoitAdam94 commented Oct 13, 2022

Finally Supermodel 3 on Github ! Keep up the good work guys !

EDIT : I am one of the users still using Legacy 3D Engine ;)

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

No branches or pull requests

3 participants