Skip to content
This repository has been archived by the owner on Oct 11, 2021. It is now read-only.

refactor: make FitParameter.value required #432

Merged
merged 1 commit into from
Jan 10, 2021

Conversation

redeboer
Copy link
Member

No description provided.

@redeboer redeboer added the ⚠️ Interface Changes to the interface label Jan 10, 2021
@redeboer redeboer self-assigned this Jan 10, 2021
@codecov
Copy link

codecov bot commented Jan 10, 2021

Codecov Report

Merging #432 (18a9648) into master (4718ec9) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #432   +/-   ##
=======================================
  Coverage   90.02%   90.02%           
=======================================
  Files          22       22           
  Lines        3437     3437           
  Branches      847      847           
=======================================
  Hits         3094     3094           
  Misses        173      173           
  Partials      170      170           
Flag Coverage Δ
unittests 90.02% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/expertsystem/amplitude/model.py 92.82% <100.00%> (ø)

@redeboer redeboer merged commit 7a98d6b into master Jan 10, 2021
@redeboer redeboer deleted the required-FitParmeter.value branch January 10, 2021 21:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
⚠️ Interface Changes to the interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant