Skip to content

修正部分内容#25

Merged
NuanRMxi merged 2 commits intoTeamFlos:mainfrom
2278535805:main
Nov 9, 2024
Merged

修正部分内容#25
NuanRMxi merged 2 commits intoTeamFlos:mainfrom
2278535805:main

Conversation

@2278535805
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@NuanRMxi NuanRMxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

除了 offset 类型的更正、notetime 的类型的更正以及对 start end start2 end2的描述更改,其他均为错误,请更正

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

holdTime有误,谱面文件中holdTime的值含有小数点。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note初始化时距离判定线的高度note打击时判定线流过的速度 意思相同

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

什么是 intt,时间单位为 float 无误,并非 int

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

time都是int? 好像是float吧

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

notetimeint ,事件的 timefloat

@2278535805
Copy link
Copy Markdown
Contributor Author

已修改

@NuanRMxi NuanRMxi merged commit 6ac8489 into TeamFlos:main Nov 9, 2024
@NuanRMxi
Copy link
Copy Markdown
Member

NuanRMxi commented Nov 9, 2024

感谢您的修正。

Mivik pushed a commit that referenced this pull request Apr 8, 2026
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.

3 participants