Skip to content

Adds more documentation and revises API slightly#15

Merged
zygmuntszpak merged 1 commit intoJuliaImages:masterfrom
zygmuntszpak:more_refactor
Nov 28, 2018
Merged

Adds more documentation and revises API slightly#15
zygmuntszpak merged 1 commit intoJuliaImages:masterfrom
zygmuntszpak:more_refactor

Conversation

@zygmuntszpak
Copy link
Copy Markdown
Member

I got rid of an illogical optical_flow! function that I added previously but didn't think through carefully enough. Added some more documentation.

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 27, 2018

Codecov Report

Merging #15 into master will not change coverage.
The diff coverage is 91.66%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #15   +/-   ##
======================================
  Coverage    64.2%   64.2%           
======================================
  Files           5       5           
  Lines         366     366           
======================================
  Hits          235     235           
  Misses        131     131
Impacted Files Coverage Δ
src/farneback.jl 58.78% <100%> (ø) ⬆️
src/optical_flow.jl 85.71% <90%> (ø) ⬆️

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 ad7d9ad...06d9bb3. Read the comment docs.

@zygmuntszpak zygmuntszpak force-pushed the more_refactor branch 4 times, most recently from 044cc5a to 84311f8 Compare November 28, 2018 09:12
@zygmuntszpak zygmuntszpak merged commit 1038682 into JuliaImages:master Nov 28, 2018
@zygmuntszpak zygmuntszpak deleted the more_refactor branch November 28, 2018 10:20
zygmuntszpak added a commit to zygmuntszpak/ImageTracking.jl that referenced this pull request Nov 29, 2018
Adds more documentation and revises API slightly
zygmuntszpak added a commit to zygmuntszpak/ImageTracking.jl that referenced this pull request Nov 29, 2018
Adds more documentation and revises API slightly
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.

1 participant