Tillerino / Tillerinobot

352 stars 112 forks source link

specify number of 100s/50s instead of acc in !acc #136

Closed omkelderman closed 7 years ago

omkelderman commented 7 years ago

Instead of giving a accuracy, give an amount of 100s and/or 50s as parameter.

The actual accuracy that will be given to the pp-calc system must then be calculated using those inputs. Keeping in mind that currently it is also possible to give number of misses in !acc.

I dont know for sure if its possible since the accuracy formula requires you to know the total number of possible hits in a song and I dont know if that information is in the tillerino-database.

@Tillerino thoughts?

edit: and ofc current functionality just keeps working, this is additional functionality ofc 😄

Tillerino commented 7 years ago

If it works for the interface, why not?

On Thu, May 18, 2017 at 2:06 PM Olle Kelderman notifications@github.com wrote:

Instead of giving a accuracy, give an amount of 100s and/or 50s as parameter.

The actual accuracy that will be given to the pp-calc system must then be calculated using those inputs. Keeping in mind that currently it is also possible to give number of misses in !acc.

I dont know for sure if its possible since the accuracy formula requires you to know the total number of possible hits in a song and I dont know if that information is in the tillerino-database.

@Tillerino https://github.com/tillerino thoughts?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Tillerino/Tillerinobot/issues/136, or mute the thread https://github.com/notifications/unsubscribe-auth/ABL9ZfzawNhgT6oI-kcCtCLDsGtI2cmsks5r7DQtgaJpZM4NfH1h .

omkelderman commented 7 years ago

well, the thing is, if someone is telling the bot an amount of 100s (and or 50s) instead of acc, bot has to calc acc by itself, and to do that it needs the total amount of possible hits (cuz acc-formula). So for now really specific question: does the bot know the hit-amount-count or whatever the crap you wanna call it for each beatmap? cuz if not we'd have to parse .osu files? It is in oppai output I think...

Tillerino commented 7 years ago

Yeah, it knows. You can see in the PercentageEstimates interface that the 50/100s are calculated from the accuracy. Instead you can add another default method on the interface, which takes the number of 50/100s and skips that step.

On Thu, May 18, 2017 at 11:18 PM Olle Kelderman notifications@github.com wrote:

well, the thing is, if someone is telling the bot an amount of 100s (and or 50s) instead of acc, bot has to calc acc by itself, and to do that it needs the total amount of possible hits (cuz acc-formula). So for now really specific question: does the bot know the hit-amount-count or whatever the crap you wanna call it for each beatmap? cuz if not we'd have to parse .osu files? It is in oppai output I think...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Tillerino/Tillerinobot/issues/136#issuecomment-302544632, or mute the thread https://github.com/notifications/unsubscribe-auth/ABL9ZZ0SOv5X5lVTsSDFTwD4ZBpRvlVbks5r7LW7gaJpZM4NfH1h .

omkelderman commented 7 years ago

gotcha! I see.

Well then this should be fairly easy to do. Now a format to parse player input. We currently have !acc 97.5 821x 1m being acc, combo, misses. For this we need to swap acc to 100s and 50s but keep combo and misses I guess. I dunno a good suffix for "100s" or "50s" that isnt numerical :stuck_out_tongue: so maybe just do this:

!acc 10 20 300x 4m being 10 100s, 20 50s, 300 combo and 4 misses. The fact that there are two whole numbers instead of acc should be enough to parse it out right?

But on the other hand its not very clear to the enduser what they are entering, but I have no idea how to do that in a sensible way...

Small sidenote: I'm guessing I can calc amount of 300s by doing getAllObjectsCount() - amount100s - amount50s right?

Tillerino commented 7 years ago

Correct.

I'd go for something like !acc 100x50 20x100 300x 4x since it is very unclear which of the two numbers is 50s and which is 100s. The command is so specific that I don't worry about it being unpractical.

omkelderman commented 7 years ago

ok, sounds good! Will implement when I find time 😛