-
Notifications
You must be signed in to change notification settings - Fork 61
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
Refactor explore_hard_task.py and explore_normal_task.py for clarity and maintainability #235
Conversation
… clarity and maintainability.
Fix static config loading and update package requirements.
Refactor module structure and validate chapter range dynamically.
Add validation for submission number range in explore_hard_task logic.
… and logic for modular structure.
注意:代码未完成且未经过完整测试 |
note: 1. [Global] and [JP] server - normal_task_challenge2-finished - normal_task_challenge2-unfinished template do not exist since no such task open
三服检查challenge已适配并测试 |
后续打算:
|
感觉强制重推可以留但确实好像没什么用,确定是要删是吗
…________________________________
From: pur1fying ***@***.***>
Sent: Friday, January 31, 2025 12:17:20 AM
To: pur1fying/blue_archive_auto_script ***@***.***>
Cc: NanBoom ***@***.***>; Author ***@***.***>
Subject: Re: [pur1fying/blue_archive_auto_script] Refactor explore_hard_task.py and explore_normal_task.py for clarity and maintainability (PR #235)
后续打算:
1. 简化普通推图配置,将普通图推图的 "强制打每一关" 和 "是否强制打到sss" 删除,用户只需指定推图区域,BAAS就可以自动检测区域内每个任务是否需要打
―
Reply to this email directly, view it on GitHub<#235 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AKNXZY3YAYTOVSCMCDPAVIL2NJGBBAVCNFSM6AAAAABWDGOIPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMRUHE2DIMZUGA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
是的,打一次没sss那很可能是配置不够了,后续没必要再打了 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hard task 的推图数据丢了 😂
@@ -85,8 +85,22 @@ def get_region_num(self, img, region, category=int, ratio=1.0): | |||
|
|||
if temp == '': | |||
return "UNKNOWN" | |||
# 不提倡返回值类型不统一 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
此处返回值不统一,这意味着调用这里必须加特判,否则很容易炸锅
hard task 的推图数据是不是忘记添加了? @pur1fying |
抱歉,没检查就提交了 |
我将同步删除hard task配置,自动检测是否需要补task或者present |
好的,我想最终效果应该是用户指只需要输入15, 16-1这样的关卡,不需要再指定挑战/sss/礼物 |
求问在explore_normal_task 中执行to_hard_event的作用是什么? |
打完一关后会解锁下一关,关卡列表会向下滑动,切到困难关这个动画就肯定结束了 |
…ng for better clarity. BREAKING CHANGE: 基本完全重构
…eadability and efficiency. 2.refactor tasklist to avoid thread conflicts
后续计划:
|
Add early return for empty tasklist in normal and hard task methods.
#234