Add landmines #171
Open
Add landmines #171
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
|
Took a quick look at this, (it's late right now I'll look at it properly tomorrow) seems relatively fine, but
|
Contributor
Author
|
Yes, a lot of things were copy-pasted from tick notes. In the end landmines probably won't look like ticknotes if they end up being added, so DefaultLandmine and DefaultCircleLandmine are mostly placeholders |
…note (+ only trigger the landmine from user input if it is the first object in the lane)
+ don't count landmines in max combo
(wasn't supposed to be in yet, oops)
i suck at git pls forgive me :(
previously it just looked at the next HitObject. while this would probably be enough for most maps, if a mapper put way too many landmines right before a note, only the closest landmine would get removed earlier. with this change, every landmines in the "Okay" timing window of the next note will get removed earlier. (idk how to explain this better)
(probably requires additional modifications at other places, but rn this doens't seem to cause any issues)
# Conflicts: # fluXis/Skinning/Custom/CustomSkin.cs # fluXis/Skinning/Default/DefaultSkin.cs # fluXis/Skinning/ISkin.cs # fluXis/Skinning/SkinManager.cs
(do the same changes as 3eb3aa2 but for landmines)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
So, I recently tried to get into mapping, and I think the game could benefit from having some kind of "landmines" notes (I chose that name because it made sense to me, but I don't mind if it gets changed). Basically, these are notes that causes the player to miss if they hit them.
This is mostly a proof of concept, therefore I don't really expect this to go into the game without being reworked a lot.
As of right now, the timing window for hitting a mine and counting it as a miss is the same as "perfect" ratings, and holding a key while a mine is over the receptors
don'tcount as a miss.Also, landmines currently have the same appearance as tick notes but with a different colour.2025-09-21.18-17-07.mp4
These could be used for memorisation maps in which notes are too spaced from each other. In some situations, it is possible to get full accuracy by just spamming quads, and adding landmine notes could fix this issue.
Spamming quads, without landmines:
without_landmines.mp4
Smapping quads, with landmines:
with_landmines.mp4
Also, some stepmania/etterna maps have landmines in them, so this could also be used for imported maps
(this isn't implemented in this PR though).In addition, this would also allow the creation of gimmick maps similar to the ones from bms (https://www.youtube.com/watch?v=PZYh8iF29NA)
Since I don't know if there is any real interest in this, I didn't really focus on code quality (also I'm not that familiar with the code base), but if there is actual interest I'm willing to make all the necessary changes and clean-ups to get this in an acceptable state.