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

Remove automatic map selection feature #19

Closed
Tracked by #1
NaughtyChas opened this issue Aug 5, 2024 · 11 comments
Closed
Tracked by #1

Remove automatic map selection feature #19

NaughtyChas opened this issue Aug 5, 2024 · 11 comments

Comments

@NaughtyChas
Copy link
Collaborator

NaughtyChas commented Aug 5, 2024

In EX & Board screen, boxes will be automatically picked / banned when song information has updated through ipc.

Remove this for better stability.

@CloneWith
Copy link
Owner

I've just removed the automatic selection feature for Board view.

Seemingly this goes fine with the EX Stage view, so I'm closing this issue.

@CloneWith
Copy link
Owner

The test match revealed the issue again, temporarily reopening.

@CloneWith CloneWith reopened this Aug 8, 2024
@CloneWith CloneWith added bug Something isn't working question Further information is requested area:Components labels Aug 8, 2024
@cdwcgt
Copy link
Collaborator

cdwcgt commented Aug 10, 2024

所以可能的问题是
我们需不需要这个功能
基于gameplay已经不在这里实现,我们还是否需要ipc来获取beatmap

最好明确一下

@NaughtyChas
Copy link
Collaborator Author

所以可能的问题是 我们需不需要这个功能 基于gameplay已经不在这里实现,我们还是否需要ipc来获取beatmap

最好明确一下

大概率是不需要了

经过测试赛发现这个功能会自动向下一张被选的图添加状态,而被添加的状态不会根据逻辑自动轮换。也就是说,如果使用ipc自动获取beatmap来做选择会导致所有图施加同一个效果 (测试下来好像都是Red Pick)。手动设定格子后,自动选图会覆盖刚才手动设定好的正确状态。

由于我们删除了右边按钮的轮换逻辑,因此这里也大概率是出现问题的

从导播的角度来看,除非能够将下一张图自动轮换逻辑重新写好,建议还是移除这个功能

选图/Ban图轮换逻辑可以来 这里 参考一下。

@CloneWith
Copy link
Owner

我们需不需要这个功能

No need. Compared to manually switching, we are unsure about these parts:

  • Whether or not all states of a block is overriden by IPC (even we've chosen to store them separately)
  • If so, whether controllers can quickly restore the last correct of a block (especially with two teams of traps, in the worst scenario)

This feature will most probably mess the whole board up, enforcing us staffs to inhale large amounts of smoke during the normal match, not to mention we know little about this.


不需要(激寒。与手动选图相比,自动选图的行为有不确定性:

  • 是否会清除格子已有的所有状态(即使我们把不同状态分开存储了)
  • 发生此情况后,导播能否迅速将格子恢复到正确的状态(最坏的情况是,需要重新布置保图+2个陷阱)

可以想见,多数情况下这个功能会让棋盘阿米诺斯,强迫全体staff在直播间吸芙蓉王,更不用说我们对一得阁拉米所知甚少了。

@CloneWith CloneWith removed question Further information is requested area:EX-Screen labels Aug 19, 2024
@CloneWith
Copy link
Owner

Have just:

  • Add token verification for Board view maps (blocked automatic actions)
  • Allow for EX automatic map selecting (we use an individual list for this)
  • Unchanged for Mappool view maps

We can have a test sometime and then close the issue when everything about this goes as intended.

@NaughtyChas
Copy link
Collaborator Author

Pinging @CloneWith for update on this issue.

Tested and everything is looking good, changing map in the matchroom will no longer trigger the automatic map-selection feature in lazer.

I believe that the automatic map-selection feature in EX Screen is intended to left in the code, however the default status of the selected map is incorrect. It uses a status similar to "Red Pick" in board screen as default:

this unknown status Red win status
image image

When "Pick" is selected on the control panel, applied status will be correct when performing auto-selection. This bug can be recreated by following these steps:

  1. Do a fresh launch of the lazer
  2. Go straight into EX screen, without selecting status on the control panel
  3. Change the map in the match room to trigger automatic map selection

Setting the default status to "Pick" once we enter the EX screen should be a viable solution.

@CloneWith
Copy link
Owner

CloneWith commented Aug 21, 2024

It uses a status similar to "Red Pick" in board screen as default.

Yes it's true, since in all views using BeatmapChoice, it starts with a "Red Pick".

Some solutions I suppose:

  • Set the default button status (-> Neutral Pick)
  • Rule out illegal choices (e.g. Pick with colors in this case)

CloneWith added a commit that referenced this issue Aug 21, 2024
This can resolve the remaining problem of #19
@CloneWith
Copy link
Owner

CloneWith commented Aug 21, 2024

For now much of the issue should be completed.
If everything is OK then this can be closed as completed.

@CloneWith
Copy link
Owner

CloneWith commented Sep 14, 2024

测?等回

Seemingly this is completed. Closing for now and create another one if problem persists.

@NaughtyChas
Copy link
Collaborator Author

New issue where
Seems this problem have not fixed for the new intro for map picks. Do you like Smoke team picked?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants