-
-
Notifications
You must be signed in to change notification settings - Fork 221
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
Tutorial Writeup - Binary Search Algorithm #125
Comments
Binary Search is already there. Do you want to add more on this topic? Path: |
If anyone wants to update / add more content in Binary Search, please leave your proposal here. |
Hey, I have been practising binary search problems on LeetCode and would like to add more content in Binary Search. Could you please assign it to me? |
@jadhav-kunal Please share your proposal here first. Like how would you change it. |
Hey, I want to work on this. I found one issue in the code that is when the number would be at the border i.e., 2^32 it would give an error called out of bounds while we would be calculating for the value of mid. There is one small change we can do in this code so it works on the border cases also. |
which line and what is your small change? |
the value we are calculating mid. mid should be start + (end-start)/2 to avoid error at border cases. |
I think you're talking about integer overflow problem. In the tutorial, the solution is written in Python which doesn't have such problem. It happens when you use languages like C++ or Java. It covers in Tips already. |
Can you please assign this to me? |
@RaghavAgarwal15 assigned. please check out contribution guide first. |
I have already checked that |
@RaghavAgarwal15 Actually the tutorial is already there. I'm expecting one would add more details and examples to explain the topic. Any language is fine and of course you need to write your own code. |
@wingkwong Hey, I have made the changes which seemed necessary to me. |
@RaghavAgarwal15 You've deleted the checklist. Your PR won't be reviewed. |
From where? |
Besides, you are adding some code to other author code block, which is not expected. Also I'm expecting more details, rather than just adding a recursive approach without any explanations. |
Unassigned due to inactivity |
please is this still available to be assigned? @wingkwong |
@wingkwong can it have the hacktoberfest label |
@Jennifer-tech if the repo is tagged with |
please assign me this issue.
The text was updated successfully, but these errors were encountered: