Add support for running CFU-Playground natively with notebook in rad-lab env #11
Conversation
- replace container base image with deeplearning image - provision eda tooling using conda - add daisy-based workflow to build compute engine image - add compute binding for cloudbuild service account - reduce image build time to ~15min
Setting the account config by running gcloud config set account ACCOUNT command and making sure that same user account is set in account config with which RAD Lab launcher has been authenticated
This reverts commit c8ba8a3.
- replace container base image with deeplearning image - provision eda tooling using conda - add daisy-based workflow to build compute engine image - add compute binding for cloudbuild service account - reduce image build time to ~15min
each experiment in a separate directory with separate tuning and visualization notebooks.
|
Apache 2.0 License check successful! |
modules/silicon_design/scripts/build/notebooks/cfuplayground/cfuplayground.md
Show resolved
Hide resolved
| @@ -60,3 +61,12 @@ cp ${PROVISION_DIR}/papermill-launcher /usr/local/bin/ | |||
| chmod +x /usr/local/bin/papermill-launcher | |||
|
|
|||
| echo "DaisySuccess: done" | |||
|
|
|||
| echo "Cloning CFU-Playground repository" | |||
There was a problem hiding this comment.
this need to happen before DaisySuccess: done
| @@ -13,14 +13,14 @@ | |||
| # See the License for the specific language governing permissions and | |||
| # limitations under the License. | |||
| channels: | |||
| - defaults | |||
There was a problem hiding this comment.
im actually not sure it was just in the cfu playground env so i ported it over
There was a problem hiding this comment.
@proppy will double check
| @@ -34,10 +34,55 @@ dependencies: | |||
| - pandas | |||
| - pymeep=*=mpi_mpich_* | |||
| - jupyterlab | |||
| - python | |||
| - litex-hub::gcc-riscv32-elf-newlib | |||
There was a problem hiding this comment.
you shoudn't need to prefix them by litex-hub if it's the first one in the list of channels.
There was a problem hiding this comment.
if we want to prefix those, then we should do it consistently for all packages.
There was a problem hiding this comment.
I'll remove these prefixes
| - intervaltree | ||
| - json-c | ||
| - libevent | ||
| - python=3.7 |
There was a problem hiding this comment.
I'll try to remove 3.7 and see if still works
| - Pillow | ||
| - intervaltree | ||
| - junit-xml | ||
| - numpy |
There was a problem hiding this comment.
any reason we don't install the conda version of those?
There was a problem hiding this comment.
will move these to above pip
| - python-constraint | ||
| - fasm | ||
| - https://github.com/chipsalliance/f4pga/archive/8c411eb74e4bb23d1ec243a1515b9bfb48e2cd83.zip#subdirectory=f4pga | ||
| - git+https://github.com/f4pga/prjxray.git@ae546d6b7648bf4df9cf63f0b25b2028b5623c43#egg=prjxray |
There was a problem hiding this comment.
aren't those already install with - litex-hub::prjxray-db in the conda section?
|
|
||
| ```python | ||
| # Change directory to design space exploration project in CFU-Playground | ||
| %cd /CFU-Playground/proj/dse_template |
There was a problem hiding this comment.
do you need to %cd so that you can import dse_framework or are there other part of the cfu flow dependent on stuff being executed from this location?
There was a problem hiding this comment.
both are correct: to import as well as execute other parts of the cfu
|
FYI @tcal-x |
Updated environment with CFU-Playground packages, repo, and other needed toolchains. Added CFU-Playground notebook after converting to markdown.