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

Tutorial Writeup - Binary Search Algorithm #125

Open
Rittika1234 opened this issue Oct 2, 2022 · 18 comments
Open

Tutorial Writeup - Binary Search Algorithm #125

Rittika1234 opened this issue Oct 2, 2022 · 18 comments

Comments

@Rittika1234
Copy link

Rittika1234 commented Oct 2, 2022

please assign me this issue.

@wingkwong
Copy link
Owner

Binary Search is already there. Do you want to add more on this topic?

Path: tutorials/basic-topics/binary-search.md

@wingkwong
Copy link
Owner

If anyone wants to update / add more content in Binary Search, please leave your proposal here.

@jadhav-kunal
Copy link

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?

@wingkwong
Copy link
Owner

@jadhav-kunal Please share your proposal here first. Like how would you change it.

@HemantModi11
Copy link

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.

@wingkwong
Copy link
Owner

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?

@HemantModi11
Copy link

the value we are calculating mid. mid should be start + (end-start)/2 to avoid error at border cases.

@wingkwong
Copy link
Owner

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.

@RaghavAgarwal15
Copy link

Can you please assign this to me?
I would love to make my first contribution ever in this issue

@wingkwong
Copy link
Owner

@RaghavAgarwal15 assigned. please check out contribution guide first.

@RaghavAgarwal15
Copy link

RaghavAgarwal15 commented Oct 2, 2023

@RaghavAgarwal15 assigned. please check out contribution guide first.

I have already checked that
Just wanted to know if I have to provide my own code?
Can it be in C++?

@wingkwong
Copy link
Owner

@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.

@RaghavAgarwal15
Copy link

@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.
I have added the recursive codes for python,java and c++ and also added few questions which seemed necessary from the my point of view fir interviews and building concepts.
I hope ill get my first contibution for the hacktoberfest.
If any change required please quote.
If this comes successful I'll be happy to contribute to more issues
Thankyou

@wingkwong
Copy link
Owner

@RaghavAgarwal15 You've deleted the checklist. Your PR won't be reviewed.

@RaghavAgarwal15
Copy link

RaghavAgarwal15 commented Oct 2, 2023

@RaghavAgarwal15 You've deleted the checklist. Your PR won't be reviewed.

From where?
I haven't deleted any files sir
I am new to this, can you please guide

@wingkwong
Copy link
Owner

When you create a PR, there is a preset template. You can remove it entirely. Here's the screenshot from another PR.

image

@wingkwong
Copy link
Owner

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.

@wingkwong
Copy link
Owner

Unassigned due to inactivity

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

5 participants