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

Problems with Turbo #58

Closed
striche2 opened this issue Mar 25, 2017 · 28 comments
Closed

Problems with Turbo #58

striche2 opened this issue Mar 25, 2017 · 28 comments

Comments

@striche2
Copy link

striche2 commented Mar 25, 2017

It appears sometime in Jan a turbo feature was added (afffa41). However the feature seems to assume that you would never remap your button I and button II. In my case, I have remapped my buttons such that what I now use as button II is the same as toggling turbo for button II. This leads to...obvious problems. I can see no way to remap the turbo buttons. I also can't seem to turn the feature off. The code "appears" to need a setting called "Turbo_Toggling" assigned to "enabled" to turn on, but I can't find any reference to a Turbo_Toggling setting in my files, suggesting there may be an error in checking whether the feature should be enabled/disabled.

@hizzlekizzle
Copy link
Contributor

hizzlekizzle commented Mar 25, 2017

Yeah, looks like there should be a core option entry that never got added (should go down here somewhere). I'll try to get that added in sometime soon, unless you want to send a PR before I get to it :)

@striche2 I added the option in my fork but I'm not sure if it actually resolves your issue. Would you be amenable to testing it? If you can't build it yourself, I can probably do it for you if you tell me your OS/arch.

@prataczak
Copy link

prataczak commented Mar 30, 2017

I just came to bring up the same thing. :) I'd rigged up some complicated toggle solution in Xpadder, and was getting ready to test it out, when I realized someone did this.

The ideal mapping for turbo toggles to me is L3 and R3. They're out of the way, easily accessible, and won't interfere with anything else. So if someone manages to make the turbo toggle buttons re-mappable, this core will be perfect in my eyes.

But I just wanted to add, whoever decided to implement this, you're awesome! In fact, I owe someone $25. :) I offered a reward for this feature a few months back when I came up with the idea of adding turbo buttons to the NEStopia core. So, whoever implemented this, PM me with your email, and I'll send you a paypal payment.

@prataczak
Copy link

I don't see a bounty link anywhere. Can I make this a bounty?

@inactive123
Copy link
Contributor

Go to bountysource, register an account, copy and paste the URL of this issue, paste it into the address bar on bountysource, click on 'Post'.

Then add money to the bounty.

@prataczak
Copy link

Cool, thanks.

@prataczak
Copy link

@hizzlekizzle would you be so kind as to build your version with the fix for me? I have Windows 10/RetroArch 1.5.0.

I would really like to see this issue fixed - for me, it's the only thing keeping the emulation for this system from being completely perfect.

@hizzlekizzle
Copy link
Contributor

@prataczak
I assumed win64: http://www.mediafire.com/file/z8yuod2ops4xdld/mednafen_pce_fast_libretro.dll

Let me know if that gets you fixed up and I'll PR it to the main repo.

@prataczak
Copy link

prataczak commented Aug 18, 2017

Thanks for the upload - yes, win64.

Your fix addresses part of the problem. There is now a 'turbo toggling' setting, so people who were having issues with the pre-defined turbo toggle buttons can simply turn it off. So it is definitely improvement over the existing build for that reason.

But what I am truly hoping for here is the ability to re-map the pre-defined turbo buttons to any key. Sometimes people need their buttons to be configured a certain way to work with their gamepad and/or personal preferences, and the current implementation of the turbo toggles (being set in stone to pre defined keys) will only work for some people/gamepads.

@ghost
Copy link

ghost commented Aug 18, 2017 via email

@prataczak
Copy link

prataczak commented Aug 18, 2017

Yes but it would be better if 'button I on/off' and 'button II' on off was not automatically hard mapped to buttons III and IV.

If you need buttons III and IV put in specific locations for 6 button mode, it might conflict with things (as the OP pointed out) if you are using 2 button mode. That's the whole issue here. They should be their own entities, and you should be able to put them wherever you want.

@ghost
Copy link

ghost commented Aug 18, 2017

ok noted. a core option for switching X/Y on/off to L3/R3 would probably do.

@prataczak
Copy link

prataczak commented Aug 18, 2017

I appreciate any help you have to offer regarding this situation, so please don't misunderstand me. :)

But again, it should be up to the user to decide where it's convenient. Some people might not even be using gamepads with L3 and R3 buttons.

The turbo toggles for 'button I turbo toggle' and 'button II turbo toggle' should appear in the quick menu/controls as their own separate entries, and you should be able to decide what button to map them to. That is the ideal solution for everyone, because the user gets to decide where it's convenient to put them.

Edit: I may have misunderstood what you were saying also, apologies if that is the case.

@ghost
Copy link

ghost commented Aug 18, 2017

ok, got something similar to what you wanted, but its on supergrafx, the code should be the same though, although i am not entirely sure if this the the "right" way of doing it(on a programming layout perspective).

https://pastebin.com/c2xkYKnE

@hizzlekizzle should be able to create a new core from that, but i think he is busy so just wait up.

@prataczak
Copy link

Thanks retro-wertz. :)

@prataczak
Copy link

@retro-wertz if your fix works I would like you to claim the bounty I posted so that it does not go to waste.

I appreciate your help regardless but the whole point of the bounty system is to reward people who solve problems, and if you've solved my problem, then you've definitely earned it. :)

@ghost
Copy link

ghost commented Aug 31, 2017

i never pushed remap for turbo on/off to L3/R3 as you can just use Save Game Remap File/Save Core Remap File for those. For games that user 4/6 buttons, i remap as needed and use Save Game Remap so that when i reload these games again, remaps are automatically set... Then use Save Core Remap for all the 2-button games as there are a whole lot more 2-button games anyways.

@prataczak
Copy link

You can't remap them currently, that's the issue here.

@prataczak
Copy link

They're hard mapped to buttons III and IV, so whatever you have III and IV set to is where your turbo buttons will be.

@ghost
Copy link

ghost commented Aug 31, 2017

in pce? are you using the individual button turbo on/off in core? or do you mean the X/Y hotkey to turn turbo on button II and I?

@prataczak
Copy link

Yes, someone implemented hotkeys, and they hard mapped them to III and IV instead of making them their own separate entries that you can map anywhere.

@ghost
Copy link

ghost commented Aug 31, 2017

i dont see a remap issue here, although i see another issue which is if you already enabled turbo using X/Y hotkey in 2-button mode, switching to 6-button does not turn off turbo for button I and II.

@prataczak
Copy link

The only way to remap the turbo hotkeys is to remap buttons III and IV. In other emulators, there would simply be a 'Turbo button I' and 'Turbo button II.'

I need III and IV where I need them. And I need turbo I and turbo II toggles elsewhere. :)

@ghost
Copy link

ghost commented Aug 31, 2017

thats where save core/save game remap file comes in, as you do not need button III and IV for 2-button games.

@prataczak
Copy link

I mean I suppose that is a solution, but ideally, you could set everything up one time, and not have to figure which games need remaps.

What is the point of having a toggle 2 button/6 button mode, if I still have to configure all the 6 button games?

I'm not trying to seem ungrateful here. I'm glad there are turbo toggles. But I offered a $25 bounty for the ability to move the turbo toggle keys independently from buttons III and IV, and that is what I'd like to see happen.

If you're not personally interested in fixing that, that's fine, I won't hold it against you. But it's still something I'd like to see implemented. :)

@prataczak
Copy link

prataczak commented Dec 2, 2017

I'm going to try one more time to present my case. I have a Hori EX Turbo controller that I use. Here is the current button mapping:

example1

As you can see, I have the buttons mapped exactly the way they would be on a real 6 button controller. However, with the current implementation of the 'turbo toggle' buttons, they are not in an ideal location. What I want is this:

example 2

In this example, the 'turbo toggle' buttons are independent of buttons III and IV. With all due respect (because I know you guys work hard), it should have been implemented this way in the first place. The best solution is a solution that allows the end user to custom configure things in a way that works for their own personal setup. That is not the case here. Here, the user is forced to use the setup given to them.

Making remap files for every 6 button game is a huge hassle. I already have the controller set up exactly the way I want it. When I'm playing a 6 button game, it should be a simple matter of pressing the 'toggle 6 button mode' button.

@retro-wertz , you did such a nice job implementing the turbo toggle buttons in the PC-FX core. Can't you please implement it the same way here, and make the turbo toggle buttons independent of buttons III and IV? I am a huge TurboGrafx/NEC fan. The current implementation is very awkward for me.

@prataczak
Copy link

I upped the bounty from $25 to $70.

@prataczak
Copy link

prataczak commented Dec 5, 2017

Edit: I'm just going to post what I posted in the forum so that it is very clear what I want exactly for the bounty.

The current implementation of the turbo toggle buttons has Turbo toggle I hard mapped to button IV, and Turbo toggle II hard mapped to button III.

Also, there is no way to turn off the toggle turbo buttons in the PCE Fast core. Previously, you could toggle the turbo buttons in the menu, but the new changes completely breaks that functionality.

The SuperGrafx core will allow you to toggle off the turbo toggles in the menu.

I upgraded the bounty from $25 to $70 for this. I play a lot of TurboGrafx and I find the current implementation to be awkward. I would like to see the PCE Fast Core have the ability to toggle off the turbo toggles (like the SuperGrafx core), and I would like to have the turbo toggles as separate buttons that are not tied to the III and IV buttons, that can be moved wherever the user chooses (for both cores).

Also, since the turbo toggles would be independent with the new implementation, I would like them to continue to work as intended even while in 6 button mode.

Current implementation disables the turbo toggles after toggling to 6 button mode (since they become buttons III and IV).

@hizzlekizzle
Copy link
Contributor

Implemented in #76

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

4 participants