Skip to content
Snippets Groups Projects

Add script to install CI versions of all external Bob dependencies

Merged Laurent COLBOIS requested to merge install-bob-deps into master
1 unresolved thread

This is a proposal for a script that would help create a local environment containing all external Bob dependencies and using the same pins as the CI.

TODO

  • Create entry point (bdt dev dependencies)
  • Update documentation (Note : the CI build is seemingly using cuda builds of the pytorch and tensorflow, which would not be the case locally unless having done an export CONDA_OVERRIDE_CUDA=x.x or running on a machine with GPU.)
  • Add compilers package to the installed list

ping @ydayer @amohammadi

Do you feel this is a sensible way to approach things ?

Edited by Laurent COLBOIS

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 1 """Create an environment with all external dependencies listed in bob/devtools/data/conda_build_config.yaml"""
    2 import click
    3
    4
    5 @click.command(
    6 epilog="""Example:
    7
    8 Creates an environment called `myenv' based on Python 3.8 and containing all external bob dependencies:
    9
    10
    11 bdt dev dependencies --python 3.8 myenv
    12 """
    13 )
    14 @click.argument("env_name", nargs=1)
  • LGTM, you can even merge as it is if you want to.

  • Laurent COLBOIS marked this merge request as ready

    marked this merge request as ready

  • added 1 commit

    • 36a211cb - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Laurent COLBOIS removed review request for @amohammadi

    removed review request for @amohammadi

  • Laurent COLBOIS enabled an automatic merge when the pipeline for 36a211cb succeeds

    enabled an automatic merge when the pipeline for 36a211cb succeeds

  • Laurent COLBOIS mentioned in commit 48fb8839

    mentioned in commit 48fb8839

  • Please register or sign in to reply
    Loading