moving NumPy into PE102 where it belongs. #30
Conversation
|
We also need a way to navigate to this file, either by editing We also need a way to render the cells when deploying to the site. We'll need numpy and matplotlib on CI. I will take care of that. |
|
Actually, Quarto will not execute the notebooks by default because "the presumption being that you have already executed them while editing the notebook". https://quarto.org/docs/projects/code-execution.html#notebooks There are execution errors in https://github.com/RENCI-NRIG/X-CITE/blob/main/theme1/PE102/PE102-02NumPy.ipynb. We should re-run the notebook locally again, fixing those errors, and commit those changes. |
|
That notebook has a dependency on matplotlib. Is that the problem?
Erik
…________________________________________
From: Sajith Sasidharan ***@***.***>
Sent: Wednesday, May 22, 2024 10:54 AM
To: RENCI-NRIG/X-CITE
Cc: Scott, Erik Lamond; State change
Subject: Re: [RENCI-NRIG/X-CITE] moving NumPy into PE102 where it belongs. (PR #30)
Actually, Quarto will not execute the notebooks by default because "the presumption being that you have already executed them while editing the notebook".
https://quarto.org/docs/projects/code-execution.html#notebooks
There are execution errors in https://github.com/RENCI-NRIG/X-CITE/blob/main/theme1/PE102/PE102-02NumPy.ipynb. We should re-run the notebook locally again, fixing those errors, and commit those changes.
—
Reply to this email directly, view it on GitHub<#30 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABOBH7LCQUVXPN6LLKYKUN3ZDSWUFAVCNFSM6AAAAABICPBNBGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRVGAYDSNJWGU>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
|
It should not be a problem if we've executed cells in the notebook locally. If we want, we can install numpy and matplotlib on CI, and get the workflows to execute them on CI. But that is not really necessary, at least right now. |
|
There was a typo in a package alias name. NumPy is already there, no problem. I took out the example with matplotlib, it was really just left over from some previous lab exercise.
…________________________________________
From: Sajith Sasidharan ***@***.***>
Sent: Wednesday, May 22, 2024 11:52 AM
To: RENCI-NRIG/X-CITE
Cc: Scott, Erik Lamond; State change
Subject: Re: [RENCI-NRIG/X-CITE] moving NumPy into PE102 where it belongs. (PR #30)
It should not be a problem if we've executed cells in the notebook locally. If we want, we can install numpy and matplotlib on CI, and get the workflows to execute them on CI. But that is not really necessary, at least right now.
—
Reply to this email directly, view it on GitHub<#30 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABOBH7NHSEJ3QEXYMOXWUZ3ZDS5KRAVCNFSM6AAAAABICPBNBGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRVGEZTMNRUG4>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
PE101 is for packages in general and conda. Moved to PE102 Numerical analysis.