Skip to content

Separate treatment of Ts and array eltype in c2d#470

Merged
baggepinnen merged 2 commits intomasterfrom
c2d-patch
Apr 27, 2021
Merged

Separate treatment of Ts and array eltype in c2d#470
baggepinnen merged 2 commits intomasterfrom
c2d-patch

Conversation

@baggepinnen
Copy link
Copy Markdown
Member

No description provided.

@JuliaControlBot
Copy link
Copy Markdown

This is an automated message.
Plots were compared to references. 11/11 images have changed, see differences below.
After pulling this PR, please update the reference images by creating a PR to ControlExamplePlots.jl here.

Difference Reference Image New Image
❌ 0.031 Reference New
⚠️ 0.025 Reference New
✔️ 0.008 Reference New
⚠️ 0.019 Reference New
⚠️ 0.024 Reference New
⚠️ 0.025 Reference New
⚠️ 0.017 Reference New
⚠️ 0.023 Reference New
❌ 0.032 Reference New
✔️ 0.006 Reference New
✔️ 0.01 Reference New

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 27, 2021

Codecov Report

Merging #470 (c0f3ff3) into master (484b00f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #470   +/-   ##
=======================================
  Coverage   85.20%   85.20%           
=======================================
  Files          31       31           
  Lines        3102     3103    +1     
=======================================
+ Hits         2643     2644    +1     
  Misses        459      459           
Impacted Files Coverage Δ
src/discrete.jl 85.71% <100.00%> (+0.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 d200134...c0f3ff3. Read the comment docs.

@baggepinnen baggepinnen merged commit 21f1ca4 into master Apr 27, 2021
@baggepinnen baggepinnen deleted the c2d-patch branch April 27, 2021 18:18
@albheim albheim mentioned this pull request Sep 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants