jeff-regier / Celeste.jl

Scalable inference for a generative model of astronomical images
MIT License
183 stars 28 forks source link

make infer_init() consistent with usage #694

Closed kbarbary closed 6 years ago

kbarbary commented 6 years ago

The use of infer_init() in ParallelRun.jl requires it to return a length-4 tuple. It was returning a length-6 tuple. This PR removes the two extra return values. (These had been added for multinode_run.jl, which no longer exists).

codecov-io commented 6 years ago

Codecov Report

Merging #694 into master will decrease coverage by 0.01%. The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #694      +/-   ##
==========================================
- Coverage   73.91%   73.89%   -0.02%     
==========================================
  Files          36       36              
  Lines        3822     3812      -10     
==========================================
- Hits         2825     2817       -8     
+ Misses        997      995       -2
Impacted Files Coverage Δ
src/ParallelRun.jl 40% <50%> (-1.25%) :arrow_down:

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 a8f420c...11995fb. Read the comment docs.