Skip to content

Parallel tests #1786

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft

Parallel tests #1786

wants to merge 15 commits into from

Conversation

johmathe
Copy link
Collaborator

@johmathe johmathe commented Jan 6, 2023 •
edited
Loading

Checklist

  • My pull request has a clear and explanatory title.
  • If necessary, my code is vectorized.
  • I added appropriate unit tests.
  • I made sure the code passes all unit tests. (refer to comment below)
  • My PR follows PEP8 guidelines. (refer to comment below)
  • My PR follows geomstats coding style and API.
  • My code is properly documented and I made sure the documentation renders properly. (Link)
  • I linked to issues and PRs that are relevant to this PR.

Description

Issue

Additional context

@johmathe johmathe requested a review from ninamiolane January 6, 2023 04:05
@johmathe johmathe force-pushed the johmathe-parallel-tests branch 2 times, most recently from f66e722 to 64486cf Compare January 6, 2023 21:23
@johmathe johmathe force-pushed the johmathe-parallel-tests branch from 64486cf to 1dbcfd3 Compare January 6, 2023 21:59
@luisfpereira
Copy link
Collaborator

@johmathe, do you have any idea why this doesn't work out?

@luisfpereira luisfpereira marked this pull request as draft June 28, 2023 09:47
@johmathe
Copy link
Collaborator Author

Hard to parse from the logs here - it probably needs a few hours of investigation to see if it's possible to reproduce locally. I'm currently off for a week so hard to dig until July 6th :)

@luisfpereira luisfpereira added the testing new or updated test label Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ninamiolane ninamiolane Awaiting requested review from ninamiolane

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
testing new or updated test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@johmathe @luisfpereira @ninamiolane
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy