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

Use payu clone in docs #221

Open
3 tasks
aekiss opened this issue Aug 30, 2024 · 5 comments
Open
3 tasks

Use payu clone in docs #221

aekiss opened this issue Aug 30, 2024 · 5 comments
Labels
documentation Improvements or additions to documentation

Comments

@aekiss
Copy link
Contributor

aekiss commented Aug 30, 2024

Documentation should be updated to use payu clone:

@aekiss aekiss added the documentation Improvements or additions to documentation label Aug 30, 2024
@anton-seaice
Copy link
Contributor

See ACCESS-NRI/access-om3-wav-configs#50 (comment) and ACCESS-NRI/access-om3-wav-configs#50 (comment) for suggestions on updates to the README.md.

@aidanheerdegen
Copy link

Re runlog

In these configs, it is false currently (or should be)

Is the intention for this to be the case into the future?

@anton-seaice
Copy link
Contributor

Re runlog

In these configs, it is false currently (or should be)

Is the intention for this to be the case into the future?

No its just for convenience.

When release time comes, this needs changing to runlog:True

(when there is a payu run --no-runlog we can probably change it also)

@aidanheerdegen
Copy link

No its just for convenience.

Whose convenience?

Just trying to get at the motivation. If we really want users to have this switched on then I think we should set it to be True by default. The downside is for devs who forget to turn it off, but the downside there is just a rebase and drop some commits. A user who forgets to turn it on can't magically make commits appear that never existed.

@aekiss
Copy link
Contributor Author

aekiss commented Sep 13, 2024

I agree this should be True when production-intent configs are released, but in the meantime False is more convenient for developers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants