added -nc (no clobber) to wget bootstrap in notebooks

the-full-stack / fsdl-text-recognizer-2022

Source of the FSDL 2022 labs, which are at https://github.com/full-stack-deep-learning/fsdl-text-recognizer-2022-labs

https://fullstackdeeplearning.com/course

MIT License

82 stars 26 forks source link

added -nc (no clobber) to wget bootstrap in notebooks #54

Closed FarisHijazi closed 2 years ago

FarisHijazi commented 2 years ago

this flag will skip bootstrap.py files if they already exist, I noticed it would be annoying to run when I have no internet because wget will time out and delete the already existing bootstrap.py files, -nc will simply skip the download and network call if the file already exists

charlesfrye commented 2 years ago

Thanks for the suggestion, @FarisHijazi!

However, removing the bootstrap.py files if they exist actually is part of the intended behavior. We update the content of that file to resolve issues, and we want to be sure that anyone running the labs is using the latest version.

That feels more important than handling the case where the system doesn't have internet access, because many other aspects of the labs (installation, media embeds, resource fetches) assume internet access.