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

Add instructions for running nmse_PSL.ipynb #261

Merged
merged 2 commits into from
Aug 9, 2024

Conversation

mnlevy1981
Copy link
Contributor

Since CUPiD will now run a notebook, I also added instructions for creating the JupyterBook (although we probably need better instructions when it comes to "how can I look at the JupyterBook")

This shouldn't be tested until after NCAR/CUPiD#121 is merged to CUPiD's main branch

Since CUPiD will now run a notebook, I also added instructions for creating the
JupyterBook (although we probably need better instructions when it comes to
"how can I look at the JupyterBook")
Students trying to open the browser on derecho will get a command-not-found
error.
@TeaganKing
Copy link

Hi @nusbaume the CUPiD PR is now merged to main, so this should be ready for review! Looks like I don't have permissions to add reviewers, but Mike said he was planning to ask you to review it since you have all of the tutorial cases, etc, already.

@mnlevy1981 mnlevy1981 marked this pull request as ready for review August 7, 2024 23:20
@mnlevy1981
Copy link
Contributor Author

Thanks @TeaganKing!

@nusbaume
Copy link
Collaborator

nusbaume commented Aug 8, 2024

Great! Testing now, and will let you know how it goes.

@nusbaume
Copy link
Collaborator

nusbaume commented Aug 9, 2024

Well I never got this fully tested (hit some random issues), but I am fine with committing it and just seeing what happens. Fingers crossed!

@nusbaume nusbaume merged commit bff1d47 into NCAR:main Aug 9, 2024
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.

3 participants