search

KARPED1EM / TownOfNext

An AmongUs mod with learning & love
https://tonx.cc
GNU General Public License v3.0
58 stars 32 forks source link

Some improvements #167

Closed NikoCat233 closed 9 months ago

NikoCat233 commented 9 months ago

1.Port anti ghostsdotasks from tohe+ 提高部分情况下的性能 目前卡顿貌似是由hudmanager里的roletext导致的 2.Add blackroom pos for fungle 在山顶 3.Horse mode set to be same with Main.horseMode 似乎没有必要进行赋值操作 4.Remove rpc set pet for dead player (AU官方在fungle更新解决了这个问题 并且rpcsetpet在大多数情况下对原版是无效的，在官方修复宠物问题前TOHE使用setpetstr发包来强制修改宠物)

NikoCat233 commented 9 months ago

自评：看不明白toh5的码字

KARPED1EM commented 9 months ago

我还是比较建议你一个东西只提一个Pr，这样的话其中几个我能合并 全放一起话，只要有一个没过，那就没法合并了

NikoCat233 commented 9 months ago

这都是短代码啊 一个修改提一个pr是不是太奢侈了

KARPED1EM commented 9 months ago

这都是短代码啊 一个修改提一个pr是不是太奢侈了

怎么会奢侈呢，顶多就是有点麻烦

KARPED1EM commented 9 months ago

现在就有个例子，我还在研究①和④的时候，我打算合并你的其他代码，但是显然这些在同一个pr甚至同一个提交里 所以当其他功能我自己写好了的时候，你才刚回复我的消息

NikoCat233 commented 9 months ago

那我先不动了 直接等你吧

KARPED1EM commented 9 months ago

好的，麻烦你了

KARPED1EM commented 9 months ago

所以这个ghostsdotasks的假选项真的能起到作用？ 我在思索游戏居然会去找模组自定义类里的Option？ 请问是否有相关的详解和测试数据？

NikoCat233 commented 9 months ago

我也不清楚这是什么原理 我自己测试的时候如果只在optionbackup里面排除ghostsdotasks，不加这个optionitem ghostsdotasks的话 控制台还是会刷version 7 dows not have setting ghostsdotasks

这个是tohe+的gurge44整的，他也没给详细数据 光看码字每次在syncsettings的时候IGameOptions BuildGameOptions() 都会读一遍预设， 然后如果没有ghostsdotasks的话就会在控制台报错

我自己主观感受是去ghostdotasks之前每次要大量同步设置的时候就会卡一下 去了以后游戏基本没什么卡顿

要数据我没得，我技术不够qaq

NikoCat233 commented 9 months ago
image

也是很好奇为什么树懒会留一个躲猫猫和原版都不用的boolen在这

KARPED1EM commented 9 months ago

我的观点更偏向于游戏不可能会去找模组的Option，毕竟这是游戏的报错不是模组的报错。就算有解决方案，也不太可能是这样的。

Gurge44的水平在我印象里不至于能把这行万年报错解决掉，倒是没别的意思，只是个人看法里感觉这个小伎俩没有必要加，等到未来有更成熟的解决方案再说吧。

KARPED1EM commented 9 months ago

此外，你这份pr的其他部分功能我已经在前几天加到dev版本了，所以到时候具体Commit里可能没你的名字，不过还是感谢你的pr

KARPED1EM commented 9 months ago

image 也是很好奇为什么树懒会留一个躲猫猫和原版都不用的boolen在这

随他去吧，AU代码屎山不是一天两天了 XD