Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve inference in several places #438

Merged
merged 1 commit into from Nov 7, 2020
Merged

Improve inference in several places #438

merged 1 commit into from Nov 7, 2020

Conversation

timholy
Copy link
Member

@timholy timholy commented Nov 7, 2020

This disabling of the debugging statements may be a bit unfortunate,
but we haven't needed them lately and they are easy to uncomment.

xref timholy/CodeTracking.jl#73

This disabling of the debugging statements may be a bit unfortunate,
but we haven't needed them lately and they are easy to uncomment.

xref timholy/CodeTracking.jl#73
@codecov
Copy link

codecov bot commented Nov 7, 2020

Codecov Report

Merging #438 (c37c168) into master (8bd596b) will increase coverage by 0.45%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #438      +/-   ##
==========================================
+ Coverage   88.91%   89.37%   +0.45%     
==========================================
  Files          12       12              
  Lines        2274     2268       -6     
==========================================
+ Hits         2022     2027       +5     
+ Misses        252      241      -11     
Impacted Files Coverage Δ
src/interpret.jl 86.65% <ø> (+0.76%) ⬆️
src/breakpoints.jl 95.02% <100.00%> (ø)
src/construct.jl 90.76% <100.00%> (ø)
src/utils.jl 89.41% <100.00%> (+1.88%) ⬆️
src/optimize.jl 97.90% <0.00%> (-0.01%) ⬇️

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 8bd596b...e058b3b. Read the comment docs.

@timholy timholy merged commit c737bac into master Nov 7, 2020
@timholy timholy deleted the teh/nowarn_37193 branch November 7, 2020 19:02
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.

None yet

2 participants