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

All test cases doesn't pass for https://www.hackerrank.com/challenges/candies #7

Closed
sarathnair opened this issue May 25, 2016 · 3 comments
Closed

All test cases doesn't pass for https://www.hackerrank.com/challenges/candies #7

sarathnair opened this issue May 25, 2016 · 3 comments

Comments

@sarathnair
Copy link

The solution you posted for /master/Candies/Solution.cpp doesn't pass test cases 11 and 12
@ehotinger
Copy link
Owner

Thanks, I'll look into it when I get a chance. Seems like they added additional test cases since I've posted the original solution. If you've already fixed the issue with the original algorithm, feel free to make a PR and I'll merge it. Also, if you know the input which makes it fail and you can post that, it would be helpful as well

@jakchod
Copy link

jakchod commented Jul 14, 2017

You need long long int for solution.

ehotinger added a commit that referenced this issue Mar 15, 2018
@ehotinger
Update Candies Solution
d661159 
They changed the test cases to be outside of the `int` range. Fixes #7
@ehotinger ehotinger mentioned this issue Mar 15, 2018
Merged
@ehotinger
Copy link
Owner

ehotinger commented Mar 15, 2018 •
edited
Loading

@jakchod finally looked at this almost a few years later ;) - yeah, they changed the problem description. Feel free to make PRs to this repo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ehotinger @sarathnair @jakchod