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

原神评分权重未生效 #761

Open
CRAZYShimakaze opened this issue Jul 6, 2024 · 8 comments
Open

原神评分权重未生效 #761

CRAZYShimakaze opened this issue Jul 6, 2024 · 8 comments

Comments

@CRAZYShimakaze
Copy link

resources/meta-gs/artifact/artis-mark.js里的部分权重没生效,采用的是resources/meta-gs/character/*/artis.js里的权重

@1umine
Copy link
Contributor

1umine commented Jul 7, 2024

resources/meta-gs/artifact/artis-mark.js 里就是默认权重呀, 有自定义的就优先采用 resources/meta-gs/character/*/artis.js 里的自定义权重了

@CRAZYShimakaze
Copy link
Author

CRAZYShimakaze commented Jul 7, 2024

resources/meta-gs/artifact/artis-mark.js 里就是默认权重呀, 有自定义的就优先采用 resources/meta-gs/character/*/artis.js 里的自定义权重了

因为看到最近的pr在改默认权重值,但是自定义里的没改,比如艾尔海森的非满命通用权重,只在默认权重里改了,自定义里没改。这个是符合预期的么。

@1umine
Copy link
Contributor

1umine commented Jul 7, 2024

因为看到最近的pr在改默认权重值,但是自定义里的没改,比如艾尔海森的非满命通用权重,只在默认权重里改了,自定义里没改。这个是符合预期的么。

艾尔海森只有通用的,没有自定义

@CRAZYShimakaze
Copy link
Author

因为看到最近的pr在改默认权重值,但是自定义里的没改,比如艾尔海森的非满命通用权重,只在默认权重里改了,自定义里没改。这个是符合预期的么。

艾尔海森只有通用的,没有自定义

说错了😂,万叶的

@1umine
Copy link
Contributor

1umine commented Jul 7, 2024

说错了😂,万叶的

万叶有自定义的评分规则,非满命万叶是用的自定义的(character下的)里的通用的,不是用 mark 那个里的,没有问题

@CRAZYShimakaze
Copy link
Author

说错了😂,万叶的

万叶有自定义的评分规则,非满命万叶是用的自定义的(character下的)里的通用的,不是用 mark 那个里的,没有问题

12aae21
这个commit里面改了mark里万叶的通用权重,按当前的逻辑实际上是没有生效的,我意思是这个应该不符合预期吧

@1umine
Copy link
Contributor

1umine commented Jul 7, 2024

12aae21 这个commit里面改了mark里万叶的通用权重,按当前的逻辑实际上是没有生效的,我意思是这个应该不符合预期吧

确实

@CRAZYShimakaze
Copy link
Author

12aae21 这个commit里面改了mark里万叶的通用权重,按当前的逻辑实际上是没有生效的,我意思是这个应该不符合预期吧

确实

嗯嗯,感觉可以优化一下这里的逻辑.

1umine added a commit to 1umine/miao-plugin that referenced this issue Jul 9, 2024
修复万叶默认评分标准被自定义通用标准覆盖的问题
SmallK111407 pushed a commit that referenced this issue Aug 28, 2024
* feature: 添加老婆
  在原来的基础上添加而非直接设置为此次指定的老婆(们)

* fix issue #761 (pr #754 中万叶通用评分标准未生效)
修复万叶默认评分标准被自定义通用标准覆盖的问题

* fix: 可能会找不到模块导入失败
* 枫原万叶这里原来的 import from 可能报错找不到模块,明明之前还可以突然就不行了(
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

No branches or pull requests

2 participants