Skip to content

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