Comments (17)
yes, it is
from bancho.py.
issue resolved in dccbc39? please close if verified resolved, or I will close if I verify myself
from bancho.py.
not resolved, it does not replace the score since it checks for newScorePP > oldScorePP and since Maniera does not have cv support, 0 > 0 == false, therefore will never replace the first score you have set
from bancho.py.
How to resolve this (without full cv calc): generally check for score on mania submission.
This will not have much drawback imo since pp is generally dependent on score in mania
Another idea is having multiple layers of "new best score assertion"
from bancho.py.
Sorry i didn't get around to this for so long - so basically it looks like the issue is convert pp on mania not working and always returning 0.00? is this also the case for other gamemodes converted pp? I can do some investigation myself as well.
from bancho.py.
Another idea is having multiple layers of "new best score assertion"
Yea we should likely have multiple layers; on very difficult maps since pp is truncated, it's possible for people to repetitively get 0.00pp even if there is a working difficulty calculator, so there should be a fallback (likely to score, and perhaps further steps beyond that)
from bancho.py.
Sorry i didn't get around to this for so long - so basically it looks like the issue is convert pp on mania not working and always returning 0.00? is this also the case for other gamemodes converted pp? I can do some investigation myself as well.
doesn't seem to be an issue on taiko or catch the beat -- purely a mania issue
from bancho.py.
i think going forward the best bet is probably not using peace performance (or maniera if we still use that, i don’t remember). we use it on akatsuki and it has caused us multiple issues on other modes and it doesn’t look like it’s being maintained anymore either. i think we should look into using https://github.com/MaxOhn/rosu-pp as it is still maintained, has up to date reworks that peace doesn’t have. it also has python bindings available via pip
from bancho.py.
does rosu-pp support converted maps?
from bancho.py.
does rosu-pp support converted maps?
yep!
from bancho.py.
it'd definitely be a good move forwards then.
eventually, i'd like to make our pp calculator use consistent - e.g. rather than using multiple different libraries/programs for pp calculation, to combine it into a single, consistent interface. the code could be simplified GREATLY from this, it's one of my pet peeves with osu server codebases as a whole at the moment.
from bancho.py.
rosu-pp is a good gateway for this as its really easy to edit (and compile), but would lose the ability of using the pip package directly. in an ideal world we wouldn’t have to have the end user do any compiling for changes but too much of a perfect world. however rosu-pp is pretty well setup for making changes which is why the new WIP akatsuki system is using it since it allows us to use a single calculator across all modes (including rx)
from bancho.py.
rosu-pp is a good gateway for this as its really easy to edit (and compile), but would lose the ability of using the pip package directly. in an ideal world we wouldn’t have to have the end user do any compiling for changes but too much of a perfect world. however rosu-pp is pretty well setup for making changes which is why the new WIP akatsuki system is using it since it allows us to use a single calculator across all modes (including rx)
The latest rosu-pp-py seems to already support installation via pip: pip install rosu-pp-py
We can try switching to rosu-pp-py, or keep using peace-performance-python, now it will also build with the latest rosu-pp code instead of peace-performance.
The latest version of peace-performance-python is now 2.0.0.
from bancho.py.
bancho.py has already switched to rosu-pp!
rosu-pp is a good gateway for this as its really easy to edit (and compile), but would lose the ability of using the pip package directly. in an ideal world we wouldn’t have to have the end user do any compiling for changes but too much of a perfect world. however rosu-pp is pretty well setup for making changes which is why the new WIP akatsuki system is using it since it allows us to use a single calculator across all modes (including rx)
The latest rosu-pp-py seems to already support installation via pip:
pip install rosu-pp-py
We can try switching to rosu-pp-py, or keep using peace-performance-python, now it will also build with the latest rosu-pp code instead of peace-performance.
The latest version of peace-performance-python is now 2.0.0.
from bancho.py.
bancho.py has already switched to rosu-pp!
rosu-pp is a good gateway for this as its really easy to edit (and compile), but would lose the ability of using the pip package directly. in an ideal world we wouldn’t have to have the end user do any compiling for changes but too much of a perfect world. however rosu-pp is pretty well setup for making changes which is why the new WIP akatsuki system is using it since it allows us to use a single calculator across all modes (including rx)
The latest rosu-pp-py seems to already support installation via pip:
pip install rosu-pp-py
We can try switching to rosu-pp-py, or keep using peace-performance-python, now it will also build with the latest rosu-pp code instead of peace-performance.
The latest version of peace-performance-python is now 2.0.0.
Currently this repository still uses peace-performance-python=1.1.2 requirements.txt and some users reported the same issue to me.
So a temporary solution is necessary
from bancho.py.
I think we can close this because it has been fixed in #315
from bancho.py.
Fixed in #315
from bancho.py.
Related Issues (20)
- Push master builds to docker hub in CI
- docs: update documentation to mention caddy / local setup with caddy
- Replace docker volumes with proper directory mounting
- misc: [06:40:35PM] [GET] 404 osu.magsec.ml/web/osu-session.php HOT 9
- misc: Add in readme to install docker desktop to make it available within ubuntu
- Support for per-mod personal best scores
- misc: spin up a temporary database for integration test suite
- misc: update docker image to not run as root HOT 1
- misc: determine a method to globally handle custom json serialization using `httpx` HOT 2
- misc: (TODO) make sure all repo `fetch_many` calls are using `page=None, page_size=None` where required by osu!
- bug: Handle case of clients marking match history as private in multi creation
- misc: Move achievements and default avatar into a submodule included with the project to avoid downloads
- misc: Create serialize() and deserialize() methods for each repo to enrich data quality HOT 2
- bug: Fix flakiness of "Test Application Code" CI job with regard to establishing mysql connection
- bug: !top doesn't work with offline players HOT 1
- bug: output for offline users in !user command is confusing HOT 1
- feat: make !recent work with offline players
- feat: make !purge to clear user plays - e.g. they are blatant/other. HOT 10
- feat: Split common nginx `server` block used for all routes into individual blocks to allow for extensibility
- bug: Leaderboard placement and personal best sort issues for pp sort HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bancho.py.