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

Remove TC1 from docs examples. #62

Merged
merged 1 commit into from
Jun 18, 2022
Merged

Remove TC1 from docs examples. #62

merged 1 commit into from
Jun 18, 2022

Conversation

jb-mackay
Copy link
Contributor

@jb-mackay jb-mackay commented Jun 17, 2022

Purpose and Content

Removes tc1 (heat equation + slab) from the docs as it now breaks. It should be replaced by the sea breeze which is cleaner, uses ClimaCoupler, and is checked by buildkite.

PR Checklist

  • This PR has a corresponding issue OR is linked to an SDI.
  • I have followed CliMA's codebase contribution and style guidelines OR N/A.
  • I have followed CliMA's documentation policy.
  • I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
  • I linted my code on my local machine prior to submission OR N/A.
  • Unit tests are included OR N/A.
  • Code used in an integration test OR N/A.
  • All tests ran successfully on my local machine OR N/A.
  • All classes, modules, and function contain docstrings OR N/A.
  • Documentation has been added/updated OR N/A.

@jb-mackay
Copy link
Contributor Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jun 18, 2022

@bors bors bot merged commit b1848bb into main Jun 18, 2022
@bors bors bot deleted the bm/docs_remove_tc1 branch June 18, 2022 07:30
@jb-mackay jb-mackay mentioned this pull request Jul 5, 2022
10 tasks
bors bot added a commit that referenced this pull request Aug 2, 2022
75: Sea Breeze Tutorial Docs r=LenkaNovak a=jb-mackay

## Purpose and Content
This PR will turn the sea breeze test case into a tutorial to be included in the docs. This replaces tc1 which was removed in #62. 

The sea breeze driver, README, and all files included in the driver should be documented and included as tutorial pages.

## Benefits and Risks
This tutorial should clearly and concisely explain and demonstrate the different parts of the ClimaCoupler interface as well as how to set up a coupled simulation. It should be straightforward to add docs as the interface is enhanced.

## Linked Issues
- Closes #63 

## PR Checklist
- [x] This PR has a corresponding issue OR is linked to an SDI.
- [x] I have followed CliMA's codebase [contribution](https://clima.github.io/ClimateMachine.jl/latest/Contributing/) and [style](https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/) guidelines OR N/A.
- [x] I have followed CliMA's [documentation policy](https://github.com/CliMA/policies/wiki/Documentation-Policy).
- [x] I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
- [x] I linted my code on my local machine prior to submission OR N/A.
- [x] Unit tests are included OR N/A.
- [x] Code used in an integration test OR N/A.
- [x] All tests ran successfully on my local machine OR N/A.
- [x] All classes, modules, and function contain docstrings OR N/A.
- [x] Documentation has been added/updated OR N/A.


Co-authored-by: Ben Mackay <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant