search

NCAR / CTSM-Tutorial

CTSM Tutorial Materials
32 stars 28 forks source link

Additional updates to Tutorial 2 - Generic Single Point #19

Closed danicalombardozzi closed 2 years ago

danicalombardozzi commented 2 years ago

Previous pull request didn't include any of my changes (forgot the initial push). This includes minor text and formatting suggestions as well as pointers for how to ensure people are in bash shell.

wwieder commented 2 years ago

I think these came in via @briandobbins changes in the last PR that was merged here?

adrifoster commented 2 years ago

I think these came in via @briandobbins changes in the last PR that was merged here?

I don't think so, these were specific comments Danica had but maybe Brian incorporated them on his own?

briandobbins commented 2 years ago

I'm admittedly unsure where all the various versions are -- I think what I'm hoping we can do this morning is go through each notebook, and have one person make changes as needed and commit to the repo. If we can end the meeting with a canonical version, we can each do some new testing of it, and aim to get this out by early afternoon.

Cheers,

On Wed, May 18, 2022 at 8:20 AM Adrianna Foster @.***> wrote:

I think these came in via @briandobbins https://github.com/briandobbins changes in the last PR that was merged here?

I don't think so, these were specific comments Danica had but maybe Brian incorporated them on his own?

— Reply to this email directly, view it on GitHub https://github.com/NCAR/CTSM-Tutorial-2022/pull/19#issuecomment-1130080020, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACL2HPIQR3MEKP5U76KGD4TVKT4A7ANCNFSM5V57EWJA . You are receiving this because you were mentioned.Message ID: @.***>