Refactor for ML discoverability #246
Conversation
bcumming
requested review from
RMeli,
boeschf,
henrique,
lukasgd and
msimberg
as code owners
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
|
preview available: https://docs.tds.cscs.ch/246 |
|
|
||
| [:octicons-arrow-right-24: Climate and Weather Platform (Santis)][ref-platform-cwp] | ||
| Alps is a general-purpose compute and data Research Infrastructure (RI) open to the broad community of researchers in Switzerland and the rest of the world. | ||
| Find out more about Alps... |
There was a problem hiding this comment.
Unsure if the ... signals an unfinished thought or leads the way to the Alps box below. I would probably just leave out this sentence?
mkdocs.yml
Outdated
| - 'Slurm': running/slurm.md | ||
| - 'Job report': running/jobreport.md | ||
| - 'Known issues': running/known-issues.md | ||
| - 'Machine Learning': |
There was a problem hiding this comment.
I would probably still put Machine Learning next to (below) Software as that's mostly where it belongs logically. I don't think moving it a few steps down makes it significantly less discoverable?
docs/ml/data.md
Outdated
| @@ -0,0 +1,6 @@ | |||
| # Data management for machine learning | |||
|
|
|||
| !!! under-construction | |||
There was a problem hiding this comment.
can't we remove these Under-construction's, merge and open a new PR for them?
also the Tutorials have to be [constantly] updated... next qpm...
LGTM otherwise, thanks a lot!
There was a problem hiding this comment.
Yup, I will remove in the next commit.
They were temporary, to give people some ideas for where we might put "future work".
|
preview available: https://docs.tds.cscs.ch/246 |
| - 'NAMD': software/sciapps/namd.md | ||
| - 'Quantum ESPRESSO': software/sciapps/quantumespresso.md | ||
| - 'VASP': software/sciapps/vasp.md | ||
| - 'Environments': |
There was a problem hiding this comment.
This is not an intuitive name to me for what is inside this section, but I know what you mean and I don't have better suggestions at the moment. I'll think about it, but this is absolutely not a blocking comment. Just highlighting in case we can collectively come up with a better name.
|
preview available: https://docs.tds.cscs.ch/246 |
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
1 similar comment
|
preview available: https://docs.tds.cscs.ch/246 |
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
This comment has been minimized.
This comment has been minimized.
|
preview available: https://docs.tds.cscs.ch/246 |
2 similar comments
|
preview available: https://docs.tds.cscs.ch/246 |
|
preview available: https://docs.tds.cscs.ch/246 |
|
@bcumming I took the liberty to apply some minor typo fixes etc. Thanks for the reorganization. I'll go ahead and merge this. For others who find that they have suggestions for further improvements after the merge, please open a PR, comment, or come discuss with us again. |
No description provided.