@shangfengh
CompareExchange(ref a, b, c) 的含义是 if (a == c) a = b; 所以读取的时候应该保证 b == c
故

Is your feature request related to a problem? Please describe.
目前更新器在进行大规模更新时较慢，因此需要加速

Describe the solution you'd like
由于现在是一个一个文件轮番下载，而文件多且小的情况下会很慢，因此建议进行多线程并行下载

Describe alternatives you've considered
预先将要下载的文件放入线程安全的队列当中，开多个线程逐个取文件下载

Additional context
无

Describe the bug
现在的 EndAllAction 的实现逻辑有误，仅仅是改变了状态这个变量的值，却没有等待人物的状态真正地改变，从而导致穿墙（多重移动）、光速修机（多重修机）等多种 bug。

以移动为例，现在的停止状态代码：

• 首先，移动线程没有真的停止，因为需要等到移动线程下一轮判断 IsMoving 后才能退出，而此时移动线程可能还在行走一步，但是，player 的状态已经改变了，所以这一刻如果有移动的指令发来，就会导致又开启了一个移动线程，这会造成双重移动。本质上的原因是，状态转移并没有真的完成真正的状态转移，没有真的停止人物的移动状态。应当是先通知人物停止移动，然后等待移动线程停止移动，最后再更改人物的移动状态为不移动允许新一轮移动。
• IsMoving 的职责是用来表征人物是否在移动的， 而不是控制人物是否该移动的，职责混乱。理论上 IsMoving 应当仅由控制移动的 MoveManager 来更改。

To Reproduce
选手群里已经有很多这种例子了（

Expected behavior
以移动为例（修机自行类比），一个解决方案如下（没好好起命名）：
给每个可移动物体加一个尽可以 atomic 访问的 bool 变量，设为 flag，默认是 false；，加一个锁 mtx
MoveObj 的时候

void MoveObj()
{
        while (load(ref flag)) { 移动 }
    lock (mtx) {
        IsMoving = false;
        save(ref flag, false);
        Monitor.PulseAll(mtx);
    }
}

void StopMoving() {
    lock (mtx) {
        if (IsMoving) {
            save(ref flag, true);
            while (IsMoving) Monitor.Wait(mtx);
        }
    }
    SetStatus();
}

上述操作完全封装到移动引擎 MoveManager 中，flag 不要对外暴露。
这样，flag 仅用于通知移动线程停止移动，而 IsMoving

Additional context
之前在微信群写的有点错误，锁错了地方，现在改一下。。。

Describe the bug

如果在比赛过程中允许观战，则智能体端可以通过观战接口，获取全局信息，从而实现作弊。不知道你们有没有对这个情况进行防御。

Expected behavior

也许可以通过观战时需要token来解决。

@DragonAura

  grpcio==1.49.1
- grpcio-tools=1.49.1
+ grpcio-tools==1.49.1

缺了个 =（

See conversions in #71.

Is your feature request related to a problem? Please describe.
现在的所有进程（Server, Client）都放在同一个 Docker 容器内运行，不利于内存限制、或者分开查看内存、CPU占用。

Describe the solution you'd like
可以通过在不同 Docker 容器内运行各个进程来解决。详情见下面的解决方案：
https://www.runoob.com/docker/docker-container-connection.html
即每一场比赛都创建一个 Network，各个 Client 连接到 Server。

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

Describe the bug
这里似乎以极小的概率报错：

Additional context
numOfDeceasedStudent 和 numOfEscapedStudent 的读写确实感觉怪怪的。

Describe the bug
目前选手等待下一帧的接口 Wait 在选手开启多线程，并且一起 Wait 的时候会出现小小的问题，可能会导致选手漏掉一些帧。

详情参见 eesast/THUAI5#107 的方法二

Expected behavior
允许漏帧（不修复了）或修复。

Additional context
其实不修复也没啥问题（x

Describe the bug
When running cpp version of players' code, the server might break down soon after the game starts. The timing of this seems to be in a random manner, sometimes not happening at all. As we're currently rewriting the entire .cpp file, we do not rule out multiple bugs in our own AI.cpp, but the info printed seems to point to a bug in Server. To ensure reproduction, please operate sevaral times.

To Reproduce
Steps to reproduce the behavior:

1. Open RunServer.cmd
2. Open RunCpp.cmd
3. Happens with or without opening RunGUIClient.cmd

Expected behavior
The server should print nothing during the course of the game ,except for new spectators.

Screenshots

and meanwhile in GUI:

Additional context
I do not know whether this is related (perhaps we wrote erroneous commands in RunGUIClient.cmd), but the graphic interface refuse to open correctly if it is opened after RunCpp.cmd. The error feedback is as follows:

However, it will run correctly if opened after RunServer.cmd and before RunCpp.cmd. This does not interfere with actual programming ,but can be a little annoying.

Describe the bug
现在 SendMessage 使用了 string 类型发送消息，但是这样带来了只支持 UTF-8 字符串，而不支持 raw bytes 的问题。

Expected behavior
决赛时或许可以修改有关接口或者增加发送 bytes 的接口，以支持相关操作。

Is your feature request related to a problem? Please describe.
现在前端代码所有的页面 / 控件全部挤在一起，这可能不利于后续的维护和修改。有时间的话可以考虑重构一下。

Describe the solution you'd like
可以考虑使用类似于选项卡的方式组织各个页面，可以参考：
https://wpf-tutorial.com/zh/68/tabcontrol/%E4%BD%BF%E7%94%A8wpf-tabcontrol/

Describe alternatives you've considered
其他更好的方式也欢迎大家开脑洞）

Additional context
这是个，饼

@DragonAura
现在处理并发安全的方式，messageQueue 和它的锁 messageMutex 是分开的，这个不是很好维护，每用一次就好写一次上锁，还有忘记写的风险。最好把 queue 和 mutex 封装一下，封装成一个类，这样会比较好，可以参考一下这个实现：https://github.com/eesast/THUAI5/blob/dev/CAPI/thuai/include/concurrent_queue.hpp
看了一下，里面的接口基本都可以直接拿来用

Is your feature request related to a problem? Please describe.
目前可以进一步推进本地客户端的开发。

Describe the solution you'd like
可以添加的功能包括：本地提交代码文件、查看编译信息、查看天梯、发起对战、观看直播、下载回放等操作

Describe alternatives you've considered

1. 与网站沟通相应的请求格式
2. 可以考虑将 WPF 与 下载器集成

Additional context
暂无。

刚刚发现目前 THUAI6 的配置文件是 %USERPROFILE%\Documents\THUAI6.json：

• %LOCALAPPDATA%：对应于 %USERPROFILE%\AppData\Local，用于存放本用户在本地计算机上的配置数据
• %APPDATA%：对应于 %USERPROFILE%\AppData\Roaming，用于存放本用户的漫游配置数据
• %ALLUSERSPROFILE%：对应于 %HOMEDRIVE%\ProgramData，用于存放所有用户的公共配置数据（一般由 Administrator 用户管理）

综上，咱们的 THUAI6 更适合存放在 %LOCALAPPDATA% 中。具体的规范是，在改文件夹下，再创建一个属于自己应用的文件夹，然后在该文件夹下存放配置数据。对于 THUAI6，一个可能的配置文件路径是：%LOCALAPPDATA%\THUAI6\config.json，例如上段代码改成如下的：

var dataDir = System.IO.Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData), "THUAI6");
if (!System.IO.Directory.Exists(dataDir))
{
    System.IO.Directory.CreateDirectory(dataDir);
}
var configPath = System.IO.Path.Combine(dataDir, "config.json");

然后将数据放在该 config.json 里（卸载时直接删除该 THUAI6 文件夹即可）。

不过鉴于 THUAI6 已经放在了 Documents 里，且临近决赛，为了选手的体验，不必更改。建议 THUAI7 的小同学可以改一下（将此 issue 遗传到 THUAI7），或者决赛结束之后你们有空也可以改一下（bushi

Describe the bug
Server 中存在潜在的死锁问题

To Reproduce
暂不清楚，但服务器上的确观察到此现象

Desktop (please complete the following information):
Ubuntu 22.04

Additional context
死锁问题是非常严重的，将直接导致比赛不能正常进行。欢迎大家讨论一下当前死锁问题的原因，解决这个问题。以及如果发现了复现该问题的方式，可以在本 issue 中提出。

@DragonAura

Python 库有枚举类型：https://docs.python.org/zh-cn/3.7/library/enum.html
使用枚举类型的话，就不需要写那一坨 Dict 来把枚举变成字符串了（Python 不是 C++，都什么年代了程序员还需要手动存元数据 x）
一个 Demo 如下：

#!/usr/bin/env python3

from enum import IntEnum

class Gender(IntEnum):
    MALE = 1
    FEMALE = 2
    OTHER = 3

def main():
    s = Gender.OTHER
    print(s)    # Output 'Gender.OTHER'
    print(str(s).split('.')[-1]) # Print 'OTHER'

if __name__ == '__main__':
    main()

目前 Unity 基础功能已经做好了，可以考虑进一步完善细节。例如：

1. 用不同的动画效果呈现人物不同的状态
2. 显示捣蛋鬼的攻击操作
3. 实现以 Unity 为回放文件的默认打开方式
4. 添加以玩家身份加入游戏的功能
5. 生成安卓端应用，功能至少包括观看回放文件，可以添加观战、玩家操作等功能
6. 更华丽的主界面
7. 支持氪金获得更炫酷的粒子效果
8. ……

@DragonAura 发现一个小问题。C++ 的 API 里面大部分用的都是 int32_t、int64_t 之类的定长整数，而少数的 API 直接用的 int，这显得不是很一致与和谐，有时间可以改改（

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• chore(deps): update dependency grpcio to v1.62.1

Ignored or Blocked

These are blocked by an existing closed PR and will not be recreated unless you click a checkbox below.

• chore(deps): update actions/setup-dotnet action to v4

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

基于访问者模式，实现对不同类对象的不同访问逻辑 @DragonAura
（里面的类名、接口名、方法名可以再重新斟酌斟酌，时间原因我起的比较草率）

interface IHumanAPI {
    // APIs
}

interface IBucherAPI {
    // APIs
}

interface IAI {
    void play(IHumanAPI&);
    void play(IBucherAPI&);
}

interface IGameTimer {
    void Start();
    void Stop();

    void StartGame(IAI&);
}

class HumanAI : IHumanAPI, IGameTimer {
    // APIs

    void StartGame(IAI& ai) {
        while (true) ai.play(*this);
    }
}

class BucherAI : IBucherAPI, IGameTimer {
    // APIs

    void StartGame(IAI& ai) {
        while (true) ai.play(*this);
    }
}

class AI : IAI {
    void play(IHumanAPI& human) {
        // do something
    }

    void play(IBucherAPI& bucher) {
        // do something
    }
}

然后 logic 可以这样写：

class Logic {
    IGameTimer& t;

    void Func() {
        auto ai = CreateAI();
        {
            t->Start();
            t->StartGame(ai);
            t->End();
        }
    }
}

Is your feature request related to a problem? Please describe.
目前队式在网站上发起对战后，并不能直接观看直播，而是需要等到游戏结束后下载回放文件看回放，这非常不方便。

Describe the solution you'd like
我们希望在本地连接网站后端，实现在本地就能在网站上发起对战，并实时观看直播。

Describe alternatives you've considered
目前第一步，我们可以先和网站组沟通，连接科协网站后端。

Additional context
暂无。

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

Location: renovate.json
Error type: Invalid JSON (parsing failed)
Message: Syntax error: expecting String near "bot"], }

Describe the bug

private int hp = 0;
public int HP
{
  get
  {
    lock (gameObjLock)
    return hp;
  }
  set
  {
    lock (gameObjLock)
    hp = value;
  }
}

private int hp = 0;
public int HP
{
  get => Interlocked.CompareExchange(ref hp, -1, -1);
  set => Interlocked.Exchange(ref hp, value);
}

在多线程中，通常要对set与get访问器加锁或采用原子操作保证安全，但是这并不能保证完全安全合理
例如HP+=3;就是显然的错误
其相当与先调用get，再调用set，中间插入set操作将导致结果错误

这样可能导致set访问器具有误导性

Expected behavior
目前我进行中的修改方法是单独编写set或add方法
clgg建议可以采用结构体/类的方式来解决

应当令HP+=3;的写法在语法上不成立或者能够安全地编译
这个问题应该并非没有先例，想知道有什么更优雅的解决方案

Additional context
本issue与这个issue相关，但你不需要查看这个issue

Is your feature request related to a problem? Please describe.
更新器目前在启动后会静默等待一段时间，而没有提示，这不是很友好

Describe the solution you'd like
可以简单做个提示框

Describe alternatives you've considered
无

Additional context
无

Is your feature request related to a problem? Please describe.

目前的选手接口目录结构：

.
|
+---CAPI
|   |   README.md
|   ...
+---PyAPI
|   |   README.md
|   ...
|
...

这样其实不算太好，如果以后多加语言的话可能还会导致根目录的子目录数爆炸，分工不甚清晰。

Describe the solution you'd like

建议是把选手接口都统一放在根目录的一个子目录（CAPI）下，不同语言的接口在 CAPI 里再分子目录

Describe alternatives you've considered

可能的一个目录结构：

.
|
+---CAPI
|   |
|   +---cpp
|   |   |   README.md
|   |   ...
|   +--- python
|   |   |   README.md
|   |   ...
|   |   README.md
|   ...
...

Additional context
Add any other context or screenshots about the feature request here.

Describe the bug

在 ProcessMessage 中：

但 tAI 的结束条件：

Expected behavior

最好保证正常退出

Additional context

个人建议是：

• 上策：不进行 detach，而是让其正常结束并 join
• 中策：保证 AILoop 置为 false 后不再访问此对象
• 下策：无视此问题，反正影响不大（）

Describe the bug

问题应该是它已经持有了 write lock，又去获取了 read lock；详情看图。

Expected behavior

此问题不该出现

Screenshots

Additional context

对应的 commit：f634fbc67a512ba5e464b6276cccbbd66b064508

Is your feature request related to a problem? Please describe.
当人物在翻窗时，为了避免可能的问题，现在允许穿模，但这可能导致更多的问题。

Describe the solution you'd like
可以用以下方法避免穿模：
当人物翻窗或出现可能穿模的情况时，将出现穿模的人物顺移：直接将该人物顺移到一个安全的地方，即短时间内不可能与任何人重叠的地方。

可以参考的做法如下：将地图分成若干格，当人物 A 将与人物 B 重叠时，暴力搜索到一个任何人物短时间内达不到、且离重叠人物 B 最近的格子，将 A 顺移到该格子处。

Is your feature request related to a problem? Please describe.
目前 WPF 的 GUI 简易调试界面是浅色主题，底色为纯白，这对深夜写代码和习惯暗色的选手不是特别友好。

Describe the solution you'd like
可以考虑增加深色主题，选手可以选择浅色或深色，或跟随操作系统深浅色设置。

Describe alternatives you've considered
不知道 WPF / UWP / WinUI3 / MAUI 都有没有现成的轮子可以用（个人感觉至少后三者是有的）

Additional context
作为未来的饼吧（

Describe the bug
在：#162 (comment) 中描述了这点，是我自己以前的锅，之后看看如何改吧。

Expected behavior
消除 data race

Additional context
Add any other context about the problem here.