azavea / raster-vision

An open source library and framework for deep learning on satellite and aerial imagery.
https://docs.rastervision.io
Other
2.1k stars 387 forks source link

Reason for removing fcn.py model (based on Kampffmeyer) #2

Closed alexklibisz closed 7 years ago

alexklibisz commented 7 years ago

It looks like you deleted the fcn.py model in this commit.

Is there any specific reason for leaving that one out entirely? I personally found it useful as a starting point.

Thanks for your work in putting together these models.

lewfish commented 7 years ago

I had poor results with that model, and decided to try ResNets since I thought they would be easier to train. You're right that I should have kept the original model around. Later, I realized I had a bug in my data generators which was probably causing the poor results. I've just started working with these kinds of models, so wouldn't take anything in this repo as the gospel truth at the moment.

alexklibisz commented 7 years ago

Ok I see. Same here in terms of just starting with FCNs, etc. I suppose the link to the commit above can serve as reference to the original network.

I'll also mention the author name and paper URL so that it might come up on Google as a starting point for others wanting to implement that model: