fastai / swiftai

Swift for TensorFlow's high-level API, modeled after fastai
Apache License 2.0
457 stars 51 forks source link

notebookToScript: enforce that the target directory exists #12

Closed eliask closed 5 years ago

eliask commented 5 years ago

This creates the target directory for notebookToScript output using the equivalent of mkdir -p via the Path lib.

Running the notebook causes errors by default since the directory is not created automatically in the context of this repo.

PS. I'm still confused about the distinction between swift/ directory in the course-v3 and the swiftai repo. It seems that both are maintained? Is this the primary place where changes should be made, or is it course-v3? Or both?

jph00 commented 5 years ago

I'm still confused about the distinction between swift/ directory in the course-v3 and the swiftai repo. It seems that both are maintained? Is this the primary place where changes should be made, or is it course-v3?

I think we're making changes here - but perhaps @sgugger can weigh in?

sgugger commented 5 years ago

The swift directory in course-v3 is frozen to work with v0.4 and the video lessons. Development will be done in fastai_dev/swift and propagated here (until we find a solution for the 11 submodules).