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

[Feature Request]: Remake "Allow all Switch controllers except Joycons to be reported as Official Pro Controllers #535" for 16.0.0 #595

Closed
spitzdano opened this issue Mar 10, 2023 · 15 comments
Labels
feature request New feature or request

Comments

@spitzdano
Copy link

Feature Description

Hoping to get a rerelease of "Allow all Switch controllers except Joycons to be reported as Official Pro Controllers #535" #535 for 16.0.0. A fantastic alternate build.

Rationale

To use N64 and SNES controllers as pro controllers for remapping buttons.

Additional Details

No response

@spitzdano spitzdano added the feature request New feature or request label Mar 10, 2023
@spitzdano spitzdano changed the title [Feature Request]: [Feature Request]: Remake "Allow all Switch controllers except Joycons to be reported as Official Pro Controllers #535" for 16.0.0 Mar 10, 2023
@Banz99
Copy link
Contributor

Banz99 commented Mar 10, 2023

Compiled it, but couldn't test it personally as I don't have my Nintendo Switch around. See if it works for you.

MissionControl-0.9.0-forced_pro_controller-a73b453.zip

@spitzdano
Copy link
Author

Works great for the N64 controller!

The SNES and SFC controllers are both not functional. They connect but then disconnect immediately though they do show as a pro controller for that brief second. Unfortunately I had only previously tested the N64 controller on 15.0.0 so I don't know if this didn't function before or is something new on 16.0.0

@spitzdano
Copy link
Author

Thanks again from compiling this!

@shmuga9
Copy link

shmuga9 commented Aug 23, 2023

Does this need to be updated for 16.1.0?
Or does the new Mission Control solve needing a new version each Switch update?

@ndeadly
Copy link
Owner

ndeadly commented Aug 23, 2023

Short answer: yes.

Any HOS/atmosphere update that requires Mission Control to be updated, will also require this to be updated. I won't be merging the PR into a release in its current form, but if @Banz99 wants to keep maintaining these unofficial builds I won't stop him.

@shmuga9
Copy link

shmuga9 commented Aug 23, 2023

Darn. I only need it for N64 controller support, but it hasn't been added into the official builds yet.

@shmuga9
Copy link

shmuga9 commented Aug 28, 2023

It doesn't matter right now as this Switch firmware update is minor, but is this something you would add to the main release at some point? What would have to change, out of curiosity?

@ndeadly
Copy link
Owner

ndeadly commented Aug 28, 2023

Yes, it's something I've wanted to add for some time. The main issue with the current PR is a design one. It uses bad abstractions requiring unneccesary code duplication and is not generic. A proper solution should allow not just for controllers to act as Pro Controllers, but encompass the broader idea of a controller potentially imitating any official controller (reporting as joycons is a popular request) and should either fit in with the existing classes, or the existing classes refactored to make that possible.

@Banz99
Copy link
Contributor

Banz99 commented Aug 28, 2023

Had a moment of spare time, I used it to update the build. Let me know if it works.

MissionControl-0.9.3-forced_pro_controller-d2d5a88.zip

@shmuga9
Copy link

shmuga9 commented Aug 28, 2023

@ndeadly Thanks for the response.

@Banz99 I get an error that I have to delete the Mission Control contents folder in atmosphere (010000000000bd00). Of course when I do then it's not recognized as a pro controller. (Switch, hetake, and atmosphere are all updated to work with 16.1.0). EDIT: I do appreciate you using your spare time. Didn't include thanks at the front in case you ignored the rest.

@spitzdano
Copy link
Author

Was just wondering if anyone tried @Banz99 upload? I'd like to upgrade firmware but don't want to lose the functionality of remapping the official N64 controller. If there is confirmation that would fantastic! Thanks.

@shmuga9
Copy link

shmuga9 commented Sep 23, 2023

I tried, but got the error above. Unless there's something else I'm supposed to do to make it work. The last (first?) time I installed Mission Control was months ago, so I don't recall if I needed some workaround.

@spitzdano
Copy link
Author

@shmuga9 confirmed. I got the same error about "010000000000bd00" when trying to load the system. It worked in 12.0.3 and confirmed the controller was recognized as a procontroller. As soon as I went to 12.1.0 the error occured. @Banz99 do you think you would be able to take a look?

@Banz99
Copy link
Contributor

Banz99 commented Sep 24, 2023

Sorry, completeley forgot about this. I may have just forgotten to update the libraries to the most recent version when I built that version. Try this one and let me know. @spitzdano @shmuga9.

MissionControl-0.9.4-forced_pro_controller-972df5e.zip

@shmuga9
Copy link

shmuga9 commented Sep 24, 2023

@Banz99 Can confirm, it works. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants