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

ScatterAlignPose3 and ArrayPartition fixes #874

Merged
merged 16 commits into from
Jul 9, 2022
Merged

ScatterAlignPose3 and ArrayPartition fixes #874

merged 16 commits into from
Jul 9, 2022

Conversation

dehann
Copy link
Member

@dehann dehann commented Jul 9, 2022

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #874 (22553ea) into master (eee1fcf) will decrease coverage by 9.53%.
The diff coverage is 2.32%.

@@            Coverage Diff            @@
##           master    #874      +/-   ##
=========================================
- Coverage   14.59%   5.06%   -9.54%     
=========================================
  Files          47      46       -1     
  Lines        2747    2743       -4     
=========================================
- Hits          401     139     -262     
- Misses       2346    2604     +258     
Impacted Files Coverage Δ
src/images/ScatterAlignPose2.jl 0.00% <0.00%> (-94.70%) ⬇️
src/plotting/ScatterAlignPlotting.jl 0.00% <ø> (-92.31%) ⬇️
src/images/Pose2AprilTag4Corners.jl 21.42% <33.33%> (-50.99%) ⬇️
src/zmq/ZmqCaesar.jl 0.00% <0.00%> (-100.00%) ⬇️
src/zmq/services/session.jl 0.00% <0.00%> (-66.34%) ⬇️
src/zmq/services/factors/factorSerialization.jl 0.00% <0.00%> (-36.37%) ⬇️
src/UserFunctions.jl 0.00% <0.00%> (-28.89%) ⬇️
... and 5 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@dehann dehann marked this pull request as ready for review July 9, 2022 07:10
@dehann
Copy link
Member Author

dehann commented Jul 9, 2022

This includes part of the work for ScatterAlignPose3 (more to come in follow up PRs)

@dehann dehann added CI Continuous Integration standardization labels Jul 9, 2022
@dehann dehann merged commit 492bd4a into master Jul 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants