Skip to content

Remove duplicate computation in bridges #986

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

Merged
merged 1 commit into from
Dec 29, 2019
Merged

Remove duplicate computation in bridges #986

merged 1 commit into from
Dec 29, 2019

Conversation

blegat
Copy link
Member

@blegat blegat commented Dec 29, 2019

Trivial speedup ^^

@blegat blegat added Submodule: Bridges About the Bridges submodule Type: Performance labels Dec 29, 2019
@codecov-io
Copy link

Codecov Report

Merging #986 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #986   +/-   ##
=======================================
  Coverage   95.26%   95.26%           
=======================================
  Files          97       97           
  Lines       10803    10803           
=======================================
  Hits        10291    10291           
  Misses        512      512
Impacted Files Coverage Δ
src/Bridges/Constraint/set_map.jl 100% <100%> (ø) ⬆️

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 c75f92c...892bb5c. Read the comment docs.

@blegat blegat added this to the v0.9.10 milestone Dec 29, 2019
@blegat blegat merged commit b123627 into master Dec 29, 2019
@odow odow deleted the bl/mapped_func branch December 30, 2019 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Submodule: Bridges About the Bridges submodule Type: Performance
Development

Successfully merging this pull request may close these issues.

2 participants