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 current runs to RMT #114

Merged
merged 1 commit into from
Nov 12, 2023
Merged

Conversation

XertroV
Copy link
Contributor

@XertroV XertroV commented Oct 22, 2023

this uses MLFeed to provide a current status of each players current run in the RMT session. This can be used to avoid skipping when a player is on a banger run.

(Note: apologies, some of these screenshots are of TMX together not RMT (I had both running) but I copy-pasted the code between them)

Information shown is: player name, current CP count, last CP time, delta to best time of that player this session. When a player finishes, their final time is shown in green or red depending on whether it was better or worse than their best time.

The asterisk indicates >= 1 cp behind.

The sorting method on players is time attack, so it will show players in order of their potential runs. The alternative is the race sorting method, where players with more CPs are near the top (so a slow player with more CPs would be above a fast player with fewer CPs).

Here are some screenshots of what it looks like (with just me testing):

fresh run
image

behind by < 1 cp
image

when behind by >= 1 cp
image

when ahead
image

end of run pb
image

end of run pb
image

end of run not pb
image

@XertroV
Copy link
Contributor Author

XertroV commented Oct 22, 2023

Some nice extra things would be showing the medal next to a finish time -- not sure if there are convenient ways to do this already or it would require new code.

@GreepTheSheep GreepTheSheep merged commit 4eba035 into GreepTheSheep:main Nov 12, 2023
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

Successfully merging this pull request may close these issues.

2 participants