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: Use Cylc Reinstall & Reload #561

Open
wants to merge 2 commits into
base: 8.2.x
Choose a base branch
from

Conversation

wxtim
Copy link
Member

@wxtim wxtim commented Oct 27, 2022

Closes #557

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.

For local reviewers have this is served up on my public html under the Cylc version label 8.1.0.dev

@wxtim wxtim added this to the 8.0.3 milestone Oct 27, 2022
@wxtim wxtim self-assigned this Oct 27, 2022
@wxtim wxtim added the content Addition or modification of documentation label Oct 27, 2022
@wxtim wxtim requested a review from MetRonnie October 28, 2022 08:21
src/tutorial/runtime/reinstalling.rst Outdated Show resolved Hide resolved
src/tutorial/runtime/reinstalling.rst Outdated Show resolved Hide resolved
src/tutorial/runtime/reinstalling.rst Show resolved Hide resolved
src/tutorial/runtime/reinstalling.rst Outdated Show resolved Hide resolved
…stall & reload

Response to Review
Co-authored-by: Ronnie Dutta <[email protected]>
@wxtim wxtim force-pushed the 20221027T1334--master--add_reinstall_tutorial branch from 4a798dd to 35ab418 Compare October 28, 2022 09:30
@wxtim wxtim requested a review from MetRonnie October 28, 2022 09:30
@oliver-sanders oliver-sanders modified the milestones: 8.0.3, 8.2.0 Jan 27, 2023
src/tutorial/runtime/reinstalling.rst Show resolved Hide resolved
src/tutorial/runtime/reinstalling.rst Outdated Show resolved Hide resolved
@oliver-sanders
Copy link
Member

cylc cat-log has been renamed cylc log

I backed off on that PR for reasons I can't quite remember at the moment, should reconsider in the future, but for the mo it's still cat-log

https://github.com/cylc/cylc-flow/blob/76762bbfc1ca83be88ed8b0e0a7224e6af520964/setup.cfg#L152

@oliver-sanders oliver-sanders modified the milestones: 8.2.0, 8.2.x Jul 17, 2023
@oliver-sanders oliver-sanders changed the base branch from master to 8.2.x August 31, 2023 14:43
@wxtim
Copy link
Member Author

wxtim commented Oct 12, 2023

Open for over a year. Branch in history and can be retrieved later.

@wxtim wxtim closed this Oct 12, 2023
@MetRonnie
Copy link
Member

...12 minutes later! I see no reason to close this as it's still valid. Just has dropped off people's radar?

@MetRonnie MetRonnie reopened this Oct 12, 2023
@wxtim
Copy link
Member Author

wxtim commented Oct 12, 2023

I backed off on that PR for reasons I can't quite remember at the moment, should reconsider in the future

@MetRonnie
Copy link
Member

I backed off on that PR for reasons I can't quite remember at the moment, should reconsider in the future

That's unrelated, just in response to a resolved comment Hilary made above

@oliver-sanders
Copy link
Member

Note: There were some offline discussions about how to integrate reload into the tutorial which weren't recorded on this ticket.

Following the training session the other day, I think we should consider bumping the "consolidating configuration" tutorial down the pecking order and develop a larger section about managing workflows, starting, stopping, reloading, vip, vr, etc.

@hjoliver
Copy link
Member

@cylc/core - what's the status of this PR now (not sure if Oliver's last comment suggests not merging this as-is)

@oliver-sanders
Copy link
Member

oliver-sanders commented Apr 30, 2024

We have recently rebooted user training at our end and are developing some ideas about where we might take the main tutorial. Ideas include:

  • Introducing clock triggers (this helps to solidify the understanding that the cycle point is just a label, only clock-triggers couple it to the real time).
  • Introducing the cylc vip and cylc vr working patterns (by taking users through the process of fixing issues on the fly as part of the main tutorial path).
  • Introducing basic runtime interventions (namely trigger, possibly edit runtime too).

In order to add things, we will have to remove things. We can't just pile on the concepts and still cram it into one day, so we may need to rejig things, especially the runtime tutorial. We could potentially merge the first two sections of the runtime tutorial as the first practical doesn't achieve much. We could also thin out the "consolidating configuration" part which is quite lengthy. Perhaps parameters might get bumped to the further topics section? We may need to do something with the date-time cycling section too as this has got quite long and is the hardest part.

We've not dropped this idea, but I think we may need a more coordinated approach to integrating it. We'll come back to it in time and let you know what we're thinking.

@oliver-sanders oliver-sanders modified the milestones: 8.2.x, 8.3.x Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Addition or modification of documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tutorial: work re-installation into the runtime tutorial
4 participants