compSPI / simSPI

Generative Models
9 stars 10 forks source link

quick fix of tem_tutorial bug #116

Closed sonyahanson closed 2 years ago

sonyahanson commented 2 years ago

Quick fix for issue #115 .

Added ctf_parameters to sim_config.yaml, which fixes the 'KeyError: 'distribution_type'' error in tem_tutorial.ipynb but this should ultimately be fixed a different way.

Checklist

Verify that your PR checks all the following items.

If some items are not checked, mark your PR as draft (Look for "Still in progress? Convert to Draft" on your PR) . Only mark the PR as "Ready for review" if all the items above are checked.

If you do not know how to address some items, reach out to a maintainer by requesting reviewers.

If some items cannot be addressed, explain the reason in the Description of your PR, and mark the PR ready for review

Description

Issue

Additional context

codecov[bot] commented 2 years ago

Codecov Report

Merging #116 (7074cd6) into master (3fb3595) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #116   +/-   ##
=======================================
  Coverage   98.39%   98.39%           
=======================================
  Files          15       15           
  Lines         744      744           
=======================================
  Hits          732      732           
  Misses         12       12           

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 3fb3595...7074cd6. Read the comment docs.