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

tutorial: sleep if not doing anything #6041

Closed
wxtim opened this issue Mar 28, 2024 · 2 comments · Fixed by #6047
Closed

tutorial: sleep if not doing anything #6041

wxtim opened this issue Mar 28, 2024 · 2 comments · Fixed by #6047
Assignees
Labels
could be better Not exactly a bug, but not ideal. doc Documentation
Milestone

Comments

@wxtim
Copy link
Member

wxtim commented Mar 28, 2024

Problem

Tutorial tasks run too fast to see in TUI and GUI, especially for the scheduling tutorials.

Proposed Solution

Add sleeps to tutorials to slow the running of early examples down.

Alternative proposed solution

Wait for task-by-task run mode setting provided as part of the skip mode work and use simulation mode instead. In my opinion this is better for learners because [TASK]run mode = simulation makes it very explicit that the task isn't running anything real.

@wxtim wxtim self-assigned this Mar 28, 2024
@wxtim wxtim added doc Documentation could be better Not exactly a bug, but not ideal. labels Mar 28, 2024
@wxtim wxtim added this to the 8.3.1 milestone Mar 28, 2024
oliver-sanders added a commit to oliver-sanders/cylc-flow that referenced this issue Apr 3, 2024
* Address feedback that the workflow runs too quickly.
* Add 2-4s sleeps to slow it down.
* Closes cylc#6041
@oliver-sanders
Copy link
Member

Putting a simple 2-4s sleep in for now so we can get this deployed before the next training session.

@wxtim, feel free to re-open if you had something fancier in mind.

@oliver-sanders oliver-sanders modified the milestones: 8.3.1, cylc-8.2.5 Apr 3, 2024
@wxtim
Copy link
Member Author

wxtim commented Apr 8, 2024

Putting a simple 2-4s sleep in for now so we can get this deployed before the next training session.

@wxtim, feel free to re-open if you had something fancier in mind.

I am fine with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
could be better Not exactly a bug, but not ideal. doc Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants