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.

Sign up for GitHub

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

Jump to bottom

fix python solution of problem 8Zf90G #1400

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix python solution of problem 8Zf90G #1400

wants to merge 3 commits into from

Conversation

onmoonno
Copy link
Contributor

@onmoonno onmoonno commented May 17, 2023 •
edited

Fixes #1399

我修改的是如下题目的 python 解法：

https://leetcode.cn/problems/8Zf90G/
(我不懂为什么把整数形改成float就能通过测试，如果大神有空的话，求赐教呀～）

通过截图如下：

Screen Shot 2023-05-16 at 11 35 30 PM

@onmoonno onmoonno closed this May 17, 2023
@onmoonno onmoonno reopened this May 17, 2023
labuladong
labuladong requested changes Jun 2, 2023
View reviewed changes
Copy link
Owner

@labuladong labuladong left a comment

Choose a reason for hiding this comment

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

你好，我看了你的修改，看样子你使用的是 python2 的解法，而仓库中给出的解法是 python3 的代码。我估计 float 的问题也是由于 python 版本不同导致的。你是否可以尝试提交 python3 的解法代码修复对应的题目呢，谢谢~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@labuladong labuladong labuladong requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug][{python}] {8Zf90G}
2 participants
@onmoonno @labuladong