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.

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

Fix map_reduce doctest, part 2 #33887

Closed
tobiasdiez opened this issue May 23, 2022 · 4 comments
Closed

Fix map_reduce doctest, part 2 #33887

tobiasdiez opened this issue May 23, 2022 · 4 comments
Labels
p: critical / 2 t: enhancement
Milestone
sage-9.7

Comments

@tobiasdiez
Copy link
Contributor

Follow up from #33834. The github workflow still sometimes fails, since the tasks are started in a different order, see eg https://github.com/sagemath/sagetrac-mirror/runs/6543006608?check_suite_focus=true

CC: @dimpase @mkoeppe @fchapoton

Component: misc

Author: Tobias Diez

Branch/Commit: 92c97c9

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/33887
@tobiasdiez tobiasdiez added this to the sage-9.7 milestone May 23, 2022
@mkoeppe
Copy link
Member

mkoeppe commented May 24, 2022

Reviewer: Matthias Koeppe

@mkoeppe
Copy link
Member

mkoeppe commented May 24, 2022

comment:2

LGTM

@tobiasdiez
Copy link
Contributor Author

comment:3

Thanks!

@vbraun
Copy link
Member

vbraun commented May 26, 2022

Changed branch from public/misc/fix_map_reduce_test_two to 92c97c9

@vbraun vbraun closed this as completed in 7c5c452 May 26, 2022
@vbraun vbraun mentioned this issue May 24, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p: critical / 2 t: enhancement
Projects
None yet
Milestone
sage-9.7
Development

No branches or pull requests
3 participants
@vbraun @tobiasdiez @mkoeppe