Skip to content

BUG: GH-35558 merge_asof tolerance error #35654

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

Merged

Conversation

ikedaosushi
Copy link
Contributor

@simonjayhawkins simonjayhawkins added this to the 1.1.1 milestone Aug 11, 2020
@simonjayhawkins simonjayhawkins added Regression Functionality that used to work in a prior pandas version Reshaping Concat, Merge/Join, Stack/Unstack, Explode labels Aug 11, 2020
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @ikedaosushi for the PR. AFAICT the added test passes without the code changes.

@ikedaosushi ikedaosushi force-pushed the BUG_GH35558_merge_asof_tolerance branch from 59a07c2 to e7f668b Compare August 12, 2020 16:37
@ikedaosushi
Copy link
Contributor Author

@simonjayhawkins Sorry, I made a mistake for the test code. It was fixed.

Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

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

Thanks @ikedaosushi lgtm ex missing comment

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

lgtm ex- @simonjayhawkins comment

simonjayhawkins
simonjayhawkins approved these changes Aug 13, 2020
@simonjayhawkins simonjayhawkins merged commit 4079505 into pandas-dev:master Aug 13, 2020
@simonjayhawkins
Copy link
Member

Thanks @ikedaosushi

@simonjayhawkins
Copy link
Member

@meeseeksdev backport 1.1.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Regression Functionality that used to work in a prior pandas version Reshaping Concat, Merge/Join, Stack/Unstack, Explode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Incompatible tolerance error in merge_asof with comaptible types.
3 participants