Kitti Datamodule

[annikabrundyn]

adding Kitti dataset and datamodule for semantic segmentation

note - this requires downloading the dataset first from this link

[pep8speaks]

Hello @annikabrundyn! Thanks for updating this PR.

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-09-23 18:45:35 UTC

[codecov]

Codecov Report

Merging #248 into master will decrease coverage by 0.57%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
- Coverage   88.31%   87.74%   -0.58%     
==========================================
  Files          70       72       +2     
  Lines        3407     3533     +126     
==========================================
+ Hits         3009     3100      +91     
- Misses        398      433      +35     

Flag	Coverage Δ
#cpu	31.92% <0.00%> (-0.71%)	⬇️
#pytest	31.92% <0.00%> (-0.71%)	⬇️
#unittests	87.00% <33.33%> (-0.55%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pl_bolts/datamodules/kitti_dataset.py	30.43% <30.43%> (ø)
pl_bolts/datamodules/__init__.py	100.00% <100.00%> (ø)
pl_bolts/utils/arguments.py	96.15% <0.00%> (ø)

---

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 c2cdad7...a0fa356. Read the comment docs.

[annikabrundyn]

Hey!
I'm trying to write a test for this datamodule but I'm not sure how to write a general test given that you have to first download the data from the website and then provide the path to the datamodule.
Any chance you could help share an example of how I should implement this test? :)

[Borda]

Can we have a test for this datamodule?