agimus-project / happypose

Toolbox and trackers for object pose-estimation. Based on the work CosyPose and MegaPose
https://agimus-project.github.io/happypose/
BSD 2-Clause "Simplified" License
22 stars 9 forks source link

Testing cosypose inference pipeline. #63

Closed petrikvladimir closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #63 (f280979) into dev (fd5c270) will increase coverage by 14.58%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             dev      #63       +/-   ##
==========================================
+ Coverage   3.52%   18.11%   +14.58%     
==========================================
  Files        156      181       +25     
  Lines      12154    14183     +2029     
==========================================
+ Hits         429     2569     +2140     
+ Misses     11725    11614      -111     
Files Changed Coverage Δ
...se/pose_estimators/megapose/src/megapose/config.py 96.15% <100.00%> (ø)

... and 68 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

petrikvladimir commented 1 year ago

I didn't realize those files were big. I agree they should not be part of the repository; download script seems a natural solution in the future. However, I would like to not postpone merging this test for a long time, so I moved those files to a separate repository on GitHub and cloned it inside the action. @nim65s, is that an acceptable workaround?

Once we have the download script ready, it can be easily updated in the action. If that is acceptable for merging, please do squash-and-merge so that big files are not kept in repo history.

nim65s commented 1 year ago

This was squashed and force-pushed in 03207c77f2e38ea5e0670ec411d9a9951ea25d9f