Skip to content

Add landmines#5

Open
orwenn22 wants to merge 3 commits intoInventiveRhythm:masterfrom
orwenn22:landmines
Open

Add landmines#5
orwenn22 wants to merge 3 commits intoInventiveRhythm:masterfrom
orwenn22:landmines

Conversation

@orwenn22
Copy link
Contributor

@orwenn22 orwenn22 commented Jan 24, 2026

this pr adds support for landmines, changes include :

  • adding a "Landmines" field to the Map model, it contains the amount of mines in the map
  • use an enum instead of a boolean value for the judgement type in score submission
  • update FallbackScoreSubmission to support landmines, NMN (no mines) mod and NLN mod
  • update CalculateScore to calculate the map's maximum achievable combo with mods applied (this is necessary because NMN and NLN will change the maximum combo)

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.

1 participant