projectglow / glow

An open-source toolkit for large-scale genomic analysis
https://projectglow.io
Apache License 2.0
262 stars 107 forks source link

docker container update #492

Closed williambrandler closed 2 years ago

williambrandler commented 2 years ago

What changes are proposed in this pull request?

Upgraded Hail version from 0.2.78 to 0.2.85 -> explicitly pinned this version

downgrading the version of numpy for the glow docker container. This is to avoid this issue here:

https://stackoverflow.com/questions/63761366/numpy-linalg-linalgerror-svd-did-not-converge-in-linear-least-squares-on-first

How is this patch tested?

ran all notebooks to ensure they still work confirmed correct version of numpy was installed with docker container projectglow/databricks-glow:1.1.2

Screen Shot 2022-02-24 at 12 05 43 PM

codecov[bot] commented 2 years ago

Codecov Report

Merging #492 (7f46aae) into master (4d8bee8) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #492   +/-   ##
=======================================
  Coverage   93.66%   93.66%           
=======================================
  Files          95       95           
  Lines        4875     4875           
  Branches      457      457           
=======================================
  Hits         4566     4566           
  Misses        309      309           

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 4d8bee8...7f46aae. Read the comment docs.