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

新增math-util依赖库 #92

Merged
merged 8 commits into from
Apr 25, 2024
Merged

Conversation

superName-w
Copy link
Contributor

No description provided.

@JiazhenBao
Copy link
Collaborator

功能审核中

@JiazhenBao
Copy link
Collaborator

1、maxDecimal、minDecimal、absDecimal编译报错

2、取舍方式roundingType具体有哪些,格式化逻辑、展示样式是什么文档中并未明确给出

3、addCeiling、addFloor向上或向下取整有误
4、subtractCeiling、subtractFloor向上、向下取整有误

5、divideCeiling、divideFloor向上、向下取整有误

6、3.2981×2.2123 = 7.2963,得出结果为7.30,末尾为0则直接被省略了是否有问题。
7、multiplyCeiling、multiplyFloor向上、向下取整有误

截图可联系小码哥提供

1、解决maxDecimal、minDecimal、absDecimal编译报错
2、优化文档补充取舍方式说明,以及格式化逻辑及效果说明和演示
3、修复addCeiling、addFloor向上或向下取整有误
4、修复subtractCeiling、subtractFloor向上、向下取整有误
5、修复divideCeiling、divideFloor向上、向下取整有误
6、修复multiplyCeiling、multiplyFloor向上、向下取整有误
@superName-w
Copy link
Contributor Author

1、maxDecimal、minDecimal、absDecimal编译报错

2、取舍方式roundingType具体有哪些,格式化逻辑、展示样式是什么文档中并未明确给出

3、addCeiling、addFloor向上或向下取整有误 4、subtractCeiling、subtractFloor向上、向下取整有误

5、divideCeiling、divideFloor向上、向下取整有误

6、3.2981×2.2123 = 7.2963,得出结果为7.30,末尾为0则直接被省略了是否有问题。 7、multiplyCeiling、multiplyFloor向上、向下取整有误

截图可联系小码哥提供

已重新提交

@JiazhenBao
Copy link
Collaborator

功能更新后续版本号升级。否则平台上不起效。麻烦升级后重新提交。

@superName-w
Copy link
Contributor Author

功能更新后续版本号升级。否则平台上不起效。麻烦升级后重新提交。

版本号已更新为1.0.2 并重新打包

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

@JiazhenBao
Copy link
Collaborator

1、Subtract、divide方法存在精度问题 2、addCustomScale、subtractCustomScale、divideCustomScale、multiplyCustomScale方法roundingType参数填写0时会报错,参数为1、2、3不会报错

Copy link
Contributor

Copy link
Contributor

@JiazhenBao
Copy link
Collaborator

功能审核中

@JiazhenBao JiazhenBao merged commit fb2414b into netease-lcap:main Apr 25, 2024
1 check passed
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