Testing cosypose inference pipeline.

[petrikvladimir]

No description provided.

[codecov]

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

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

[nim65s]

Thanks for this !

tests/data/crackers_example/models/obj_000002.ply is ~1MB and tests/data/crackers_example/models/obj_000002.png is ~10MB, maybe it could be downloaded with the datasets ?
This might require additional work on the download scripts and a few hosting / mirroring operations, but it can be worth it, and Alice is already looking into this part I think

[petrikvladimir]

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]

Ok for the temporary workaround, thanks !

[nim65s]

This was squashed and force-pushed in 03207c7