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

Nightly can be checked thrice in certain cases #85

Open
hellow554 opened this issue May 12, 2020 · 5 comments
Open

Nightly can be checked thrice in certain cases #85

hellow554 opened this issue May 12, 2020 · 5 comments
Labels
A-bisection-algorithm Area: The bisection algorithm bug Something isn't working

Comments

@hellow554
Copy link
Contributor

hellow554 commented May 12, 2020

$ cargo-bisect-rustc --end=2020-05-09
installing nightly-2020-05-09
RESULT: nightly-2020-05-09, ===> Yes
uninstalling nightly-2020-05-09

installing nightly-2020-05-07
RESULT: nightly-2020-05-07, ===> Yes
uninstalling nightly-2020-05-07

installing nightly-2020-05-05
RESULT: nightly-2020-05-05, ===> Yes
uninstalling nightly-2020-05-05

installing nightly-2020-05-03
RESULT: nightly-2020-05-03, ===> No
uninstalling nightly-2020-05-03

installing nightly-2020-05-05
RESULT: nightly-2020-05-05, ===> Yes
uninstalling nightly-2020-05-05

installing nightly-2020-05-04
RESULT: nightly-2020-05-04, ===> No
uninstalling nightly-2020-05-04

searched toolchains nightly-2020-05-03 through nightly-2020-05-05
installing nightly-2020-05-05
uninstalling nightly-2020-05-05


********************************************************************************
Regression in nightly-2020-05-05
********************************************************************************

As you can see, 2020-05-05 is downloaded and tested thrice. Is that necessary?

@spastorino
Copy link
Member

I've noticed this and it's one of the things that should be fixed :).

@chrissimpkins
Copy link
Member

I always run with start and end dates and have not seen many of these recent issues with end only runs. We need to look into that area of the source.

@chrissimpkins
Copy link
Member

I suggest that we look into the skip intervals when we investigate this. I sense that we may be adding far too many steps to find the first non-fail date before the bisection starts in an average case situation. I wonder if it is faster to initially step back by, say, one month and begin there?

@hellow554
Copy link
Contributor Author

Funny enough this still isn't fixed as of today:

installing nightly-2020-12-08
std for x86_64-unknown-linux-gnu: 24.58 MB / 24.58 MB [================================================================================] 100.00 % 5.77 MB/s testing...
RESULT: nightly-2020-12-08, ===> Yes
uninstalling nightly-2020-12-08

installing nightly-2020-12-01
std for x86_64-unknown-linux-gnu: 24.52 MB / 24.52 MB [================================================================================] 100.00 % 5.49 MB/s testing...
RESULT: nightly-2020-12-01, ===> No
uninstalling nightly-2020-12-01

installing nightly-2020-12-08
std for x86_64-unknown-linux-gnu: 24.58 MB / 24.58 MB [================================================================================] 100.00 % 6.96 MB/s testing...
RESULT: nightly-2020-12-08, ===> Yes
uninstalling nightly-2020-12-08

Would it be would it be feasible to save the tested toolchains together with a status in a hashmap or similar?

@hellow554
Copy link
Contributor Author

The even happens, when your start and end date are next to each other:

 cargo-bisect-rustc --start 2020-10-26 --end 2020-10-27

installing nightly-2020-10-26
RESULT: nightly-2020-10-26, ===> No
uninstalling nightly-2020-10-26

installing nightly-2020-10-27
RESULT: nightly-2020-10-27, ===> Yes
uninstalling nightly-2020-10-27

searched toolchains nightly-2020-10-26 through nightly-2020-10-27
installing nightly-2020-10-27
uninstalling nightly-2020-10-27

@ehuss ehuss added bug Something isn't working A-bisection-algorithm Area: The bisection algorithm labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-bisection-algorithm Area: The bisection algorithm bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants