search

GeoscienceAustralia / dea-orchestration

4 stars 1 forks source link

Install dea-proto odc ui's #126

Closed santoshamohan closed 5 years ago

santoshamohan commented 5 years ago

Following are the changes implemented within this PR:

  1. In raijin_scripts/deploy/build_environment_module.py file:

    • Add --no-warn-script-location flag to ignore pip install path for odc packages. pip PATH used is from dea-env whereas pip path is expecting the dea bin path to be added to PATH
    • Update code to install dea-proto odc.* packages

  2. In raijin_scripts/deploy/dea/datacube.conf file:

    • Add config setting for c3 samples

  3. In raijin_scripts/deploy/dea/modulespec.yaml file:

    • Add dea-proto pip install paths

  4. In raijin_scripts/test_deaenv/run file:

    • Remove redundant logging
santoshamohan commented 5 years ago

Thanks, Damien. I will make a note of your comments and group pip install commands, based on the pip arguments.

codecov[bot] commented 5 years ago

Codecov Report

Merging #126 into master will not change coverage. The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #126   +/-   ##
=======================================
  Coverage   60.32%   60.32%           
=======================================
  Files          16       16           
  Lines        1124     1124           
=======================================
  Hits          678      678           
  Misses        446      446
Impacted Files Coverage Δ
raijin_scripts/deploy/build_environment_module.py 20.33% <0%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 735063e...841f4d0. Read the comment docs.