-
Notifications
You must be signed in to change notification settings - Fork 6
Open
Description
Include https://github.com/rigetti/forest-tutorials here and deprecate that repository. Why?:
- They serve nearly the same purpose, but the latter builds on this repo with a suite of jupyter notebook files that could easily be kept here
- This extra layer adds more work in propagating updates, fixes, and other changes
- The dependencies for
forest-tutorialsare installed ... from arequirements.txtfile in this repo.
This was likely set up like this for one of two reasons:
- Binder uses a
Dockerfileas "configuration of last resort" and so this layering prevents Binder from seeing anything but the Dockerfile at theforest-tutorialslevel - Prevent "bloat" of extra notebooks in the
forest-notebookdocker image ... in reality, this is negligible compared to the overall size of the image. There already is a more-minimal image below this one,rigetti/forest, and having two layers on top of that is unnecessary.
As a confirmation of the rationale here, the Readmes for the two repos are quite similar, mostly overlapping in content. that suggests they serve a single purpose.
Metadata
Metadata
Assignees
Labels
No labels