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

Nix the attempt flag #651

Closed
wlandau opened this issue Jan 4, 2019 · 1 comment
Closed

Nix the attempt flag #651

wlandau opened this issue Jan 4, 2019 · 1 comment
Assignees
@wlandau
Labels
topic: performance

Comments

@wlandau
Copy link
Member

wlandau commented Jan 4, 2019

Because we call outdated() early on to trim down the schedule, we can easily check if all targets are already up to date. No need for all those silly cache reads/writes.
@wlandau wlandau self-assigned this Jan 4, 2019
wlandau pushed a commit that referenced this issue Jan 5, 2019
@wlandau-lilly
Merge pull request #652 from ropensci/schedule_experiment
abc03a1 
Fix #650 and #651
@wlandau
Copy link
Member Author

wlandau commented Jan 5, 2019

Fixed via #652.

@wlandau wlandau closed this as completed Jan 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wlandau wlandau

Labels
topic: performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wlandau