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

Add Global Brawl Netplay Settings #2276

Merged
merged 1 commit into from Apr 22, 2015
Merged

Add Global Brawl Netplay Settings #2276

merged 1 commit into from Apr 22, 2015

Conversation

jefflongo
Copy link
Contributor

Creates an ini for Super Smash Bros. Brawl and adds a gecko code for standardized tournament play (unlock all characters and stages, set to a timed, stock battle, items off, ect.)

More info here: http://smashboards.com/threads/brawl-gets-an-optimized-netplay-codeset.397646/

@JosJuice
Copy link
Member

JosJuice commented Apr 8, 2015

The file should have the following at the top: # RSBE01 - Super Smash Bros. Brawl

@pauldacheez
Copy link
Contributor

Whatever you did, the line you added disappeared along with the second commit. Might I recommend git commit --amend in this case instead of your current squashing method? http://git-scm.com/docs/git-commit
git rebase -i and marking a second commit as fixup (not squash, ironically) would work fine as well. http://git-scm.com/docs/git-rebase

@lioncash
Copy link
Member

So is this all good now?

@neobrain
Copy link
Member

I guess there should be some description in the ini as to what all that is actually about.

@jefflongo
Copy link
Contributor Author

@neobrain I'm just following the naming of Global Melee Netplay Settings. This code removes tripping, sets default items off, timed stock match, all stages/characters unlocked. Basically tournament defaults. Is there another way you'd like me to do it? The idea is to have everything in one code.

@JosJuice
Copy link
Member

Having all of that information in the name of the code probably wouldn't fit well, but I suppose you could put it in a comment (starting with #) between [Gecko] and $Global Brawl Netplay Settings. It won't be visible in the GUI, but everyone who opens the file will see it.

@jefflongo
Copy link
Contributor Author

I will go ahead and do that then.

@Stevoisiak
Copy link
Contributor

@dolphin-emu-bot rebuild

@neobrain
Copy link
Member

Sorry if I was being unclear before. It's good that the patches are now described, but I actually meant to ask for comments on what Global Brawl actually is. That's not really self-evident without having to ask Google about that.

@jefflongo
Copy link
Contributor Author

Is this better? Basically this code is just using default settings that a standard tournament would use. I'm not sure how to make it clearer from there.

Creates an ini for Super Smash Bros. Brawl and adds a gecko code for standardized tournament play (unlock all characters and stages, set to a timed, stock battle, items off, ect.)

More info here: http://smashboards.com/threads/brawl-gets-an-optimized-netplay-codeset.397646/

Update RSBE01.ini
@JMC47
Copy link
Contributor

JMC47 commented Apr 21, 2015

lgtm now based on the description. It says that it's more or less a tool to make netplay easier, but it's also pretty clear that it does other things that could be used offline for tournaments. Any other complaints.

@jefflongo jefflongo changed the title Add Global Brawl Netplay Settings Add Global Brawl Netplay Settings [do not merge yet] Apr 22, 2015
@jefflongo
Copy link
Contributor Author

we're going to decide on whether or not tripping disabled should be included or not so please don't merge this yet.

@jefflongo
Copy link
Contributor Author

Alright we're leaving it like this, this is good to go.

@jefflongo jefflongo changed the title Add Global Brawl Netplay Settings [do not merge yet] Add Global Brawl Netplay Settings Apr 22, 2015
@JMC47
Copy link
Contributor

JMC47 commented Apr 22, 2015

@dolphin-emu-bot rebuild

comex added a commit that referenced this pull request Apr 22, 2015
Add Global Brawl Netplay Settings
@comex comex merged commit 5309ab1 into dolphin-emu:master Apr 22, 2015
@ghost
Copy link

ghost commented Jun 2, 2015

Why the hell was this added? This is breaking Brawl modes. I can't pause matches, I can't rotate the camera, I can't even test my Brawl hacks properly now. Is there no way around this? Can't I disable this?

@Stevoisiak
Copy link
Contributor

@therandomizer85 It was added as an optional cheat code. These settings are not enabled by default

@ghost
Copy link

ghost commented Jun 2, 2015

Well, I can't disable it no matter what I do and it's breaking the Brawl modes. I can't remove it from the RSBE01.txt, I can't disable it as Dolphin re-enables it. This is bullshit, I'm sorry, but it's screwing the game up as I dont' even go online.

I was just testing dolphin-master-4.0-6490-x64 and the cheat code cannot be disabled on my end, at all. Can't I delete it from the ini at all?

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

Who's the fucking genius who enabled it by default? Can confirm @therandomizer85's issue, and due to the Global Settings bug, it can't be turned off.

@ghost
Copy link

ghost commented Jun 2, 2015

@JMC47 See? I'm not the only one it's happening to.

To the people of this discussion, I will personally make a video showing you all what's going on.

Edit: Not enabled by default my ass.

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

Actually, could you record a video anyway? Showing off the global settings bug would be useful for an issue report to get that fixed.

@ghost
Copy link

ghost commented Jun 2, 2015

Yes, doing so right now.

@Stevoisiak
Copy link
Contributor

@therandomizer85 Sorry, my mistake. Thanks for noticing the issue.

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

Actually it won't be useful, this Pull Request is just outright wrong.

@ghost
Copy link

ghost commented Jun 2, 2015

@Stevoisiak So, what can be done about it? Should I post the video anyway?

@ghost
Copy link

ghost commented Jun 2, 2015

I really hope that this doesn't become a problem, I don't know why the fucking thing doesn't let me turn it off.

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

Delete the whole file out of the root of your Dolphin.exe /sys/GameSettings/RSBE01.ini

@ghost
Copy link

ghost commented Jun 2, 2015

@JMC47 I have too many important codes that I can't find anywhere else and would hate to lose them. Unrestricted camera codes, size modification codes, etc, that you can't find elsewhere.

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

back it up, but I don't think your codes will be in that file. They should be in My Documents/Dolphin Emulator/GameSettings/RSBE01.ini. If they aren't, then, yeah, this is a problem.

Either way, this is a serious issue and we will look into it immediately.

I assumed windows, if you're on Linux or Mac, look in the appropriate system directories, of course.

@ghost
Copy link

ghost commented Jun 2, 2015

Isn't that there the Gecko/Ocarina codes are stored as well? And yes, the cheats are where they should be.

And yes, I deleted the actual .ini and not where the cheats are, seemed to have worked. Gonna test it, but yeah, it's a problem.

Edit: Derp, Found it and it's gone.

Edit 2: On Windows 7 Pro

There needs to be a disclaimer, warning or something. @Stevoisiak I wasn't trying to call anyone out, please accept my apologies.

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

Okay, here's a bigger problem. This PR is fine. It turns out the last Gecko Code in every file is being assumed on for some reason. This is a huge problem. Sorry @jefflongo for assuming it was this PR's fault; my lack of knowledge with gecko codes fueled that.

So, we need to find a big fix for this soon, thank you so much for bringing it to our attention.

@ghost
Copy link

ghost commented Jun 2, 2015

@JMC47 Dammit, I didn't mean to bring up what's going to be a royal pain in the ass to fix. Maybe I shouldn't have brought it up? I mean, you're welcome, you really are, but dayum, I didn't know it was gonna be such a PITA :(

@JMC47
Copy link
Contributor

JMC47 commented Jun 2, 2015

It's a serious issue that needs fixing! An oversight like this could have caused worse issues. There are other games having issues with this that we found when looking things up. You saved us trouble overall.

@ghost
Copy link

ghost commented Jun 2, 2015

I really hope I did, but ugh, I wish I knew how to program, I mean, I didn't even know it was locking up games, like, at all, but yikes. Does a thread need to be made to get the other developers' attention?

@comex
Copy link
Contributor

comex commented Jun 2, 2015

So nobody noticed this issue for 5 weeks?

I will look at it tomorrow.

probably.

@ghost
Copy link

ghost commented Jun 2, 2015

@comex Hey, I just noticed it today, and yeah, maybe I shouldn't have played Brawl today, would that have been better? Hell, maybe I shouldn't have even bothered to report it -_-

Maybe I would've been better off if I didn't report the sodding thing, I'm sure. Am I wrong? Fairly sure you guys are hella busy as it is without some no-name end user reporting things.

Edit: Sorry, it's just I can't help but imagine my trivial reports add stress. I even took the time to go through all the GIT revisions to seek red flag and lo and behold, I found it.

@comex
Copy link
Contributor

comex commented Jun 2, 2015

I'm glad you reported it, since otherwise it would probably have stayed unfixed for who knows how many more weeks. ;p

@ghost
Copy link

ghost commented Jun 2, 2015

Well, I guess it's about time I did something right for the Dolphin team for once then, heh :3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
8 participants