jtr13 / cc22mw

Community contributions for EDAV Fall 2022 Mon/Wed
https://jtr13.github.io/cc22mw/
0 stars 67 forks source link

Valid installation of TensorFlow not found #78

Open philkim72 opened 2 years ago

philkim72 commented 2 years ago

Hi @eklavyaj ,

It looks like one of your merges from earlier this afternoon failed due to "Valid installation of TensorFlow not found."

https://github.com/jtr13/cc22mw/actions/runs/3499213242

I tried addting 'rstudio/tensorflow' to the remote section, but that didn't seem to help.

jtr13 commented 2 years ago

We don't have it set up in GitHub Actions to install Python so Python code is not going to work. The simplest solution would be to set eval=FALSE to the code chunks. We could try to get Python working but it might not be so easy (for example, see: https://stackoverflow.com/questions/62547923/error-installing-miniconda-on-github-actions-in-an-r-project)

eklavyaj commented 2 years ago

Okay professor! I’ll edit that myself and commit the changes.

On Fri, Nov 18, 2022 at 9:16 PM Joyce Robbins @.***> wrote:

We don't have it set up in GitHub Actions to install Python so Python code is not going to work. The simplest solution would be to set eval=FALSE to the code chunks. We could try to get Python working but it might not be so easy (for example, see: https://stackoverflow.com/questions/62547923/error-installing-miniconda-on-github-actions-in-an-r-project )

— Reply to this email directly, view it on GitHub https://github.com/jtr13/cc22mw/issues/78#issuecomment-1320753331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDTMWQXGLHMAV6E7KSNLG3WJAZ6TANCNFSM6AAAAAASFBGQDY . You are receiving this because you were assigned.Message ID: @.***>

-- Regards

Eklavya Jain

lauradls commented 2 years ago

Hi @eklavyaj - Could you also remove the file from the bookdown in the meantime? That way we can upload the other contributions.

jtr13 commented 2 years ago

A suggestion: It may be easier to keep track of problematic chapters if you comment them out in _bookdown.yml rather than remove it.

lauradls commented 2 years ago

Thank you!

philkim72 commented 2 years ago

Hi @eklavyaj - It doesn't seem you have edited anything yet since I don't see any new actions since around 10pm last night.

eklavyaj commented 2 years ago

Yeah, I opened it to make changes right now. Looks like you already edited the bookdown.

philkim72 commented 2 years ago

I commented out r_neural_network.Rmd from the bookdown so that we can start to merge other pull requests.

eklavyaj commented 2 years ago

Understood, I'll set eval = False for all cells meanwhile.

eklavyaj commented 2 years ago

Hi @philkim72 and @jtr13, I set the chunks option eval to FALSE globally for file 'r_neural_network', but still, it seems that there is an error in building the bookdown. Can you please help me with this?

philkim72 commented 2 years ago

See post #62 under issues.  Phil Kim917-576-5065Sent from my iPhoneOn Nov 27, 2022, at 5:11 PM, Eklavya Jain @.***> wrote: Hi @philkim72 and @jtr13, I set the chunks option eval to FALSE globally for file 'r_neural_network', but still, it seems that there is an error in building the bookdown. Can you please help me with this?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

jtr13 commented 2 years ago

It's working now... the last few fails all mentioned the html_dependency issue which has to do with leaflet. It may be that some of the chapters in which you set eval=FALSE would work now, but TensorFlow will not -- it requires a Python installation that we're not set up for.