Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add jumpSearch algorithm #75

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

Closed
rubysdeadname wants to merge 3 commits into knaxus:master from rubysdeadname:master
Closed

Add jumpSearch algorithm #75

rubysdeadname wants to merge 3 commits into knaxus:master from rubysdeadname:master

Conversation

Copy link
Contributor

@rubysdeadname rubysdeadname commented Oct 11, 2019

No description provided.

Copy link
Member

TheSTL commented Oct 11, 2019
edited
Loading

Hello @gabrielbarker thanks for the PR.

  • We are not getting your commit properly . Please setup you git properly.
    image

  • And can you add time complexity for the algorithm?

ashokdey reacted with thumbs up emoji

Copy link
Contributor Author

rubysdeadname commented Oct 12, 2019
edited
Loading

Hi @TheSTL @ashokdey , sorry about that! I have added time complexity and will make a new PR to see if that solves the problem.

Copy link
Member

TheSTL commented Oct 12, 2019

I'm closing this PR.
Fixed PR can be found here [#78 ]

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

@TheSTL TheSTL Awaiting requested review from TheSTL

Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /