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

Release v2.0 的一些想法 #248

Closed
18 of 25 tasks
MistEO opened this issue Jun 6, 2024 · 21 comments · Fixed by #324
Closed
18 of 25 tasks

Release v2.0 的一些想法 #248

MistEO opened this issue Jun 6, 2024 · 21 comments · Fixed by #324
Labels
enhancement New feature or request whimsy A bold and amazing idea

Comments

@MistEO
Copy link
Member

MistEO commented Jun 6, 2024

肝不完了,剩下的 2.1 再说(

@MistEO MistEO added the whimsy A bold and amazing idea label Jun 6, 2024
@MistEO

This comment was marked as outdated.

@dongwlin
Copy link
Member

dongwlin commented Jun 6, 2024

enabled, times_limit 作为next属性,diff该怎么写,难道要重写一长串next(?)

@MistEO
Copy link
Member Author

MistEO commented Jun 6, 2024

enabled, times_limit 作为next属性,diff该怎么写,难道要重写一长串next(?)

我想到了这点,所以我删了这条()

@MistEO MistEO linked a pull request Jun 6, 2024 that will close this issue
@neko-para
Copy link
Contributor

timeout直接移除了?

@skrlst
Copy link

skrlst commented Jul 16, 2024

移除 timeout_next, runout_next 等使流程变得混乱的字段

表示 timeout_next 还是很好用的

@MistEO
Copy link
Member Author

MistEO commented Jul 16, 2024

timeout 的思路有问题,对于用户来说就是看着脚本卡了 20 秒,我觉得这种体验不好。应该更多的增加中间过程以及异常情况识别

@MistEO MistEO changed the title Pipeline v2.0 的一些想法 Release v2.0 的一些想法 Jul 17, 2024
@MistEO MistEO pinned this issue Jul 17, 2024
@neko-para
Copy link
Contributor

ProjectInterface目前来看复杂度有限, 感觉没必要作为库, 只需要规定一个spec让需要使用的自己实现就行吧
如果库只提供执行能力, 那就需要倒过来嵌入配置, 而且无法获取中间状态(比如, 单纯需要resource/controller)
如果库提供比较完整的编辑处理能力, 库本身的复杂度又过高了, 不如直接实现吧

@moomiji
Copy link
Contributor

moomiji commented Jul 17, 2024

允许 Pipeline 属性字段的默认值全局改变

@neko-para
Copy link
Contributor

我还是觉得干脆点直接用字符串来作为controller的type比较好(
不管是用StringListBuffer还是用个啥逗号分隔之类的 🤔

@SweetSmellFox
Copy link
Contributor

没有timeout_next的话,假如任务过程遇到类似于明日方舟闪退或者是明日方舟闪断更新这种情况的话该怎么处理

@moomiji
Copy link
Contributor

moomiji commented Aug 3, 2024

is sub 改成任务异常,并考虑全局异常

@MistEO
Copy link
Member Author

MistEO commented Aug 3, 2024

没有timeout_next的话,假如任务过程遇到类似于明日方舟闪退或者是明日方舟闪断更新这种情况的话该怎么处理

添加一个全局异常处理机制

@weinibuliu
Copy link
Contributor

我用is_sub主要是为了方便复用任务而不是异常处理,删掉之后配置next与catch(?)好像会有点麻烦的样子orz

@bamboo98
Copy link

bamboo98 commented Aug 4, 2024

pipeline支持图形化节点编程(连连看)
强烈要求保留timeout_next, runout_next,最起码保留到3.0再移除(

@neko-para
Copy link
Contributor

pipeline支持图形化节点编程(连连看)

这个你应该一个具体的开发辅助工具下面提,fw又没有官方编辑器

@moomiji
Copy link
Contributor

moomiji commented Aug 8, 2024

小测了下 Android API Level 的兼容性:

  • 26 及以上,MaaFramework 的行为具有一致性
  • 23 及以上,MaaTouch 可用
  • 21 及以上,MaaFramework 功能正常
  • 18 & 19,获取截图功能正常,仅 ScreencapEncodeToFile 可用,pipeline 运行会持续报错(见下文)
  • 17 及以下,截图为空图,不排除测试环境问题或修改 controller_config 可用
18 & 19
[2024-07-08 09:46:33.633][ERR][Px7237][Tx46592][UnitBase.cpp][L65][std::optional<std::string> MaaNS::CtrlUnitNs::UnitBase::startup_and_read_pipe(const ProcessArgv &, std::chrono::seconds)] child return error [argv.exec=/Users/runner/Library/Android/sdk/platform-tools/adb] [argv.args=["-s","emulator-5554","exec-out","screencap -p"]] 
[2024-07-08 09:46:33.633][ERR][Px7237][Tx46592][ControlUnitMgr.cpp][L290][bool MaaNS::CtrlUnitNs::ControlUnitMgr::_screencap(cv::Mat &)] failed to screencap 
[2024-07-08 09:46:33.633][ERR][Px7237][Tx46592][GeneralControllerAgent.cpp][L92][virtual std::optional<cv::Mat> MaaNS::ControllerNS::GeneralControllerAgent::_screencap()] controller screencap failed 
[2024-07-08 09:46:33.633][ERR][Px7237][Tx46592][ControllerAgent.cpp][L538][bool MaaNS::ControllerNS::ControllerAgent::handle_screencap()] controller screencap failed 
[2024-07-08 09:46:33.633][ERR][Px7237][Tx41195][PipelineTask.cpp][L220][std::optional<PipelineTask::HitDetail> MaaNS::TaskNS::PipelineTask::find_first(const std::vector<std::string> &)] Image is empty 

@overflow65537
Copy link
Contributor

overflow65537 commented Aug 19, 2024

roi_target能加入到2.0中吗
通过继承上一个程序的识别结果来作为roi
主要的使用场景通常是以某个图片或者文字为基准,需要点击特定偏移量的按钮。
但是由于文字或者图片大小缘故,无法准确定位按钮位置。
所以需要先定位目标图片,在添加偏移量来作为roi,最后识别特定的按钮

@MistEO MistEO linked a pull request Aug 23, 2024 that will close this issue
@MistEO MistEO removed a link to a pull request Aug 23, 2024
@MistEO MistEO added the enhancement New feature or request label Sep 7, 2024
@MistEO MistEO removed a link to a pull request Sep 7, 2024
@MistEO MistEO linked a pull request Sep 7, 2024 that will close this issue
@MistEO
Copy link
Member Author

MistEO commented Sep 8, 2024

timeout_next 保留兼容性 51d1227 (#324)

@MistEO MistEO unpinned this issue Sep 13, 2024
@Yebolin
Copy link

Yebolin commented Oct 2, 2024

StartApp

启动 App 。

该动作属性需额外部分字段:

package: string
启动入口。必选。
需要填入 package name 或 activity ,例如 com.hypergryph.arknights 或com.hypergryph.arknights/com.u8.sdk.U8UnityContext `

这个可以支持更多的参数么?
例如对于mumu有应用分身的,如果想启动分身的应用需要指定启动的用户 am start --user 10 -n ACTIVITY,这样就可以不多开模拟器实现2个(最多好像是9个)应用的共存,尤其是对于不支持账号切换的app(例如森空岛)的多账号支持。

@neko-para
Copy link
Contributor

StartApp

启动 App 。

该动作属性需额外部分字段:

package: string
启动入口。必选。
需要填入 package name 或 activity ,例如 com.hypergryph.arknights 或com.hypergryph.arknights/com.u8.sdk.U8UnityContext `

这个可以支持更多的参数么?
例如对于mumu有应用分身的,如果想启动分身的应用需要指定启动的用户 am start --user 10 -n ACTIVITY,这样就可以不多开模拟器实现2个(最多好像是9个)应用的共存,尤其是对于不支持账号切换的app(例如森空岛)的多账号支持。

这个可以考虑单独开个feature request的issue

@MistEO
Copy link
Member Author

MistEO commented Oct 2, 2024

StartApp
启动 App 。
该动作属性需额外部分字段:
package: string
启动入口。必选。
需要填入 package name 或 activity ,例如 com.hypergryph.arknights 或com.hypergryph.arknights/com.u8.sdk.U8UnityContext `

这个可以支持更多的参数么? 例如对于mumu有应用分身的,如果想启动分身的应用需要指定启动的用户 am start --user 10 -n ACTIVITY,这样就可以不多开模拟器实现2个(最多好像是9个)应用的共存,尤其是对于不支持账号切换的app(例如森空岛)的多账号支持。

我觉得这种需求可以用 custom 解决(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request whimsy A bold and amazing idea
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants