Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Seg #940

Closed
wants to merge 12 commits into from
Closed

Seg #940

wants to merge 12 commits into from

Conversation

chunhuizhang
Copy link

No description provided.

This folder contains the examples of image segmentation in MXNet.

## Sample results
![fcn-xs pasval_voc result](https://github.com/tornadomeet/mxnet/blob/seg/example/fcn-xs/fcn-xs_pascal.jpg)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This url should point to the image included in this PR

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can commit image to dmlc/web-data to reduce repo size

@vchuravy
Copy link
Contributor

Thanks :) I added some comments, but somebody else should probably review this too.

I would suggest that when you address the comments you rebase and squash the commits so that it doesn't have any merge (or otherwise superflous) commits.

@piiswrong
Copy link
Contributor

Good work! Think this is a really good example. It illustrates how to write an DataIter for custom data and how to write custom solver for custom model.

We have got a lot of questions for how to do this. It would be great if you can summarize your work into a tutorial on how to feed data and how to do custom bind and solve.
Also, given that you had to write a new solver from scratch, what do you think about the _train_multi_device refactoring here #909? Maybe you can refactor your solver to use ExecutorManager and improve the interface in the process?

@tornadomeet
Copy link
Contributor

hi, all, this examples changed some code of mashadow and mxnet specificly for seg , so there is some style we should debat,. btw, the code is some dirty, i'll rearrange the sample in this weekend~ so please wait for me some days, thanks.

@tqchen
Copy link
Member

tqchen commented Dec 16, 2015

close this PR till @tornadomeet 's update

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants