search

DagsHub / papers-with-everything

Machine Learning papers with code, data and models
28 stars 10 forks source link

Added Directory for End-to-End Optimization of Scene Layout #40

Closed Bharat-mtr closed 1 year ago

Bharat-mtr commented 1 year ago

This is regarding Issue #36 Added Directory on End-to-End Optimization of Scene Layout

Paper :- https://arxiv.org/pdf/2007.11744 Repository:- https://dagshub.com/Bharat-mtr/3D_SLN

Fixes #36

Bharat-mtr commented 1 year ago

@nirbarazida Please review and let me know if I need to add/edit anything

nirbarazida commented 1 year ago

Hey @Bharat-mtr,

Thanks for another great contribution.

As I reviewed the repo on DagsHub, I see that the data directory only holds the metadata about the SUNCG dataset and not the dataset itself. As I further investigated, it seems that the dataset is no longer available due to legal issues. As mentioned in the linked issue, it can be replaced with the 3DFront dataset, which holds images in the same format but with a different file structure.

Because the 3DFront dataset is under a license that doesn't allow to share it with 3rd parties, I think our best option here is to modify the README file and explain that the SUNCG dataset is no longer available, and link to the repo the 3DFront dataset.

Bharat-mtr commented 1 year ago

Hey @Bharat-mtr,

Thanks for another great contribution.

As I reviewed the repo on DagsHub, I see that the data directory only holds the metadata about the SUNCG dataset and not the dataset itself. As I further investigated, it seems that the dataset is no longer available due to legal issues. As mentioned in the linked issue, it can be replaced with the 3DFront dataset, which holds images in the same format but with a different file structure.

Because the 3DFront dataset is under a license that doesn't allow to share it with 3rd parties, I think our best option here is to modify the README file and explain that the SUNCG dataset is no longer available, and link to the repo the 3DFront dataset.

Thanks for pointing out. I have updated the README accordingly. Please review it.

Thanks.

Bharat-mtr commented 1 year ago

i have made the required changes @nirbarazida @deanp70 , please review it & merge it

deanp70 commented 1 year ago

Thanks for the contribution @Bharat-mtr!