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

Added test for issue #6002 (currently fails) #6003

Closed
wants to merge 11 commits into from

Conversation

RubendeBruin
Copy link

  • Closes #xxxx
  • Tests added

@max-sixty
Copy link
Collaborator

Great! If you add an xfail we can merge...

@RubendeBruin
Copy link
Author

Hi @max-sixty , what is a xfail ?

Also, the issue turned out to be bottleneck-bug , ref: pydata/bottleneck#393 (comment) (fix available)

@max-sixty
Copy link
Collaborator

Hi @max-sixty , what is a xfail ?

https://docs.pytest.org/en/6.2.x/skipping.html — it marks the test as "expected failure".

Also, the issue turned out to be bottleneck-bug , ref: pydata/bottleneck#393 (comment) (fix available)

Great that it's fixed upstream! We can either wait for it to be released and then merge this, or merge this now with the xfail

@RubendeBruin
Copy link
Author

I've added xfail , thanks for the link.

@max-sixty
Copy link
Collaborator

@RubendeBruin it seems my guidance was bad — the test failures suggest that because this is a segfault xfail doesn't work, because the whole program crashes.

So instead, let's leave this open until bottleneck does a release, and then merge this as a passing test — does that work on your end?

@RubendeBruin
Copy link
Author

"does that work on your end?"

yes it does. Will remove the xfail and then we can merge once pydata/bottleneck#382 is merged.

Ruben de Bruin and others added 4 commits November 29, 2021 09:50
@jhamman
Copy link
Member

jhamman commented Sep 14, 2023

closing as stale / upstream issue. Feel free to reopen if there is more to do here.

@jhamman jhamman closed this Sep 14, 2023
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.

4 participants