Skip to content

feat: expose client#61

Open
GuGuMur wants to merge 3 commits intoMooncellWiki:mainfrom
GuGuMur:master
Open

feat: expose client#61
GuGuMur wants to merge 3 commits intoMooncellWiki:mainfrom
GuGuMur:master

Conversation

@GuGuMur
Copy link

@GuGuMur GuGuMur commented Oct 10, 2025

No description provided.

continue

tg.start_soon(check_and_run_task, task(client), diff)
return client
Copy link
Member

Choose a reason for hiding this comment

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

这个暴露出来是用来干什么的

Copy link
Author

Choose a reason for hiding this comment

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

装置处理需要读 anon 里的 prefab ,暴露出来节省一点操作

Copy link
Contributor

Choose a reason for hiding this comment

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

虽然但是 不该在Ptilopsis_Bot搞吗(

Copy link
Author

Choose a reason for hiding this comment

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

因为我之前装置处理就是fork下来改成这样用的()

Copy link
Member

Choose a reason for hiding this comment

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

因为我之前装置处理就是fork下来改成这样用的()

直接构造 client 吧,这个 main 给 cli 用的,或者就暴露个函数来构造 client + init

Copy link
Member

Choose a reason for hiding this comment

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

装置处理具体处理的是什么,为什么不能靠task做?

Copy link
Author

Choose a reason for hiding this comment

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

  1. 调torappu是因为torappu开源前自己写的逻辑很丑,发现开源了就直接拿来用了
  2. 实体类型、阵营、阻挡半径、重写地块、再部署策略、占用部署数、部署条件、撤回策略都是读prefab的 突然理解hjhk为什么当初不直接给鸮bot补功能了

Copy link
Member

@StarHeartHunt StarHeartHunt Oct 19, 2025

Choose a reason for hiding this comment

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

  1. 调torappu是因为torappu开源前自己写的逻辑很丑,发现开源了就直接拿来用了
  2. 实体类型、阵营、阻挡半径、重写地块、再部署策略、占用部署数、部署条件、撤回策略都是读prefab的 突然理解hjhk为什么当初不直接给鸮bot补功能了

那搞成task直接这个pr merge进来呗,让llm重写一下

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