Skip to content

Cityscape data loader is not implemented - WIP #5

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

Closed
ibadami opened this issue Jul 18, 2017 · 9 comments
Closed

Cityscape data loader is not implemented - WIP #5

ibadami opened this issue Jul 18, 2017 · 9 comments

Comments

@ibadami
Copy link
Contributor

ibadami commented Jul 18, 2017

In your wiki, it states that cityscape dataset loader is implemented but it is not in the repo :)
Thank you for the rest of the code though 👍

@meetps
Copy link
Owner

meetps commented Jul 19, 2017

Ah, yes. Since I've ported this repo from an internal private torch repo, I thought that I already had ported the cityscapes loader.

Will do that over the weekend.

@amiltonwong
Copy link

Hi, @meetshah1995 ,

Have you released the cityscapes loader?

Thanks!

@balloch
Copy link

balloch commented Aug 25, 2017

Pinging this. Cityscapes loader?

@acgtyrant
Copy link

The longest weekend in history.

@cattaneod
Copy link

Any update on this?

@meetps
Copy link
Owner

meetps commented Sep 23, 2017

@acgtyrant - Finding a job while doing your thesis takes time no? :)

I've been busy with my thesis, I'm not planning to write a loader for cityscapes, feel free to send in a PR.

The fastest way IMO, to do it would be to simply port the one here to the format pytorch-semseg uses.

@meetps meetps changed the title Cityscape data loader is not implemented Cityscape data loader is not implemented - WIP Sep 23, 2017
@acgtyrant
Copy link

acgtyrant commented Sep 24, 2017

I wrote the loader for cityscapes myself already, but my company may does not agree I open its source.

I will advice the loader dependence on cityscapesScripts, generate the new trainLabel image via cityscapesscripts.preparation.json2labelImg.json2labelImg in __getitem__ and return it as the target.

By the way, the official vision is planning extending the transforms, so following this new proposal is better, as just adding the new attribute transforms to the Dataset class.

@achaiah
Copy link

achaiah commented Sep 24, 2017

In case you guys haven't found it yet, this project has a working cityscapes implementation (as well as VOC).

@meetps
Copy link
Owner

meetps commented Nov 11, 2017

Added here too.

@meetps meetps closed this as completed Nov 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants