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

qsieve with time=True doesn't work #28696

Closed
embray opened this issue Nov 5, 2019 · 3 comments
Closed

qsieve with time=True doesn't work #28696

embray opened this issue Nov 5, 2019 · 3 comments

Comments

@embray
Copy link
Contributor

embray commented Nov 5, 2019

When running something like qsieve(n, time=True), for some reason it uses the built-in time command in the shell to do this (is there really any advantage in that?)

However, on Cygwin, there is not a time program by default; it is a bash shell built-in, and there is no wrapper executable for it, so trying to run time with Popen fails as reported here: https://ask.sagemath.org/question/48549/qsieve-error/

A workaround would be to just use timeit, like

sage: timeit('qsieve(n)', repeat=1, number=1)

Component: porting: Cygwin

Keywords: qsieve cygwin time

Issue created by migration from https://trac.sagemath.org/ticket/28696

@embray embray added this to the sage-9.0 milestone Nov 5, 2019
@embray
Copy link
Contributor Author

embray commented Jan 6, 2020

comment:1

Ticket retargeted after milestone closed

@embray embray modified the milestones: sage-9.0, sage-9.1 Jan 6, 2020
@mkoeppe
Copy link
Member

mkoeppe commented May 1, 2020

comment:2

Moving tickets to milestone sage-9.2 based on a review of last modification date, branch status, and severity.

@mkoeppe mkoeppe modified the milestones: sage-9.1, sage-9.2 May 1, 2020
@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Oct 24, 2020
@mkoeppe
Copy link
Member

mkoeppe commented May 10, 2021

comment:4

Moving to 9.4, as 9.3 has been released.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 May 10, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Aug 22, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 18, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 3, 2022
@mkoeppe mkoeppe removed this from the sage-9.7 milestone Aug 25, 2022
@mkoeppe mkoeppe closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants