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

Add shift_left and shift_right algorithms #5466

Merged
merged 8 commits into from
Sep 3, 2021

Conversation

Jedi18
Copy link
Contributor

@Jedi18 Jedi18 commented Jul 25, 2021

Added shift_left and shift_right algorithms.

Any background context you want to provide?

Closes #3706
Issue #4822

@jenkins-cscs
Copy link
Collaborator

Can one of the admins verify this patch?

@hkaiser
Copy link
Member

hkaiser commented Jul 26, 2021

add to whitelist

@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch 4 times, most recently from 899f989 to 675ebd4 Compare August 3, 2021 14:10
@Jedi18 Jedi18 marked this pull request as ready for review August 3, 2021 14:10
@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch from 675ebd4 to 3ad8eea Compare August 20, 2021 08:05
@hkaiser hkaiser added this to the 1.8.0 milestone Aug 31, 2021
hkaiser
hkaiser previously approved these changes Aug 31, 2021
Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Sorry this took so long.

@hkaiser
Copy link
Member

hkaiser commented Aug 31, 2021

@Jedi18 same here: could you please fix the conflict so we can go ahead and merge the PR?

@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch 2 times, most recently from df5f806 to cba0a27 Compare September 1, 2021 04:29
@msimberg
Copy link
Contributor

msimberg commented Sep 2, 2021

@Jedi18 could you please fix the formatting errors? You can ignore the inspect errors. They've been fixed on master.

@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch from cba0a27 to f93cdab Compare September 2, 2021 11:14
@Jedi18
Copy link
Contributor Author

Jedi18 commented Sep 2, 2021

Sorry didn't notice those. Fixed the formatting errors.

@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch from f93cdab to 770133a Compare September 2, 2021 14:17
@Jedi18 Jedi18 force-pushed the add_shift_algorithms branch from 770133a to d1c3e0f Compare September 3, 2021 03:23
@hkaiser hkaiser merged commit 1983483 into STEllAR-GROUP:master Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add shift_left and shift_right algorithms
4 participants