satellite-image-deep-learning / techniques

Techniques for deep learning with satellite & aerial imagery
Apache License 2.0
8.49k stars 1.45k forks source link

Techniques and sub-sections numbered #53

Closed MuhammedM294 closed 1 year ago

MuhammedM294 commented 1 year ago

This commit only numbered the techniques list and the classification sub-section. If it's okay and approved, I will continue with other sub-sections.

robmarkcole commented 1 year ago

Fantastic, I think it is sufficient to number the techniques only, unless there is a good reason to number each resource as well?

MuhammedM294 commented 1 year ago

Thanks! Actually numbering the sub-sections is the real motivation for doing this. So, one can know during exploring the repo which paper or code belongs to which technique without the need to go back to the heading. It's only one mouse scroll and you get lost in the repo :) Besides, it will save time for future edits/issues. You'll be able to localize the item to be edited easily.

robmarkcole commented 1 year ago

OK I understand now :) Yes please complete the remaining sections for the PR, thanks!

MuhammedM294 commented 1 year ago

This branch is ready to be merged now. It's a bit less aesthetically, but more practical, and organized, and will definitely save much time for future issues.

robmarkcole commented 1 year ago

@MuhammedM294 there is an issue with Seg section - numbered 2.1, not 2

image
MuhammedM294 commented 1 year ago

Yes, I did that intentionally because there are many sub-lists of segmentation resources such as segmentation - land use & land cover and segmentation - vegetation, crops & crop boundaries. So, treated Segmentation as a sub-section of the whole technique. There is another solution that also applies to objection detection and super-resolution techniques.

  1. Segmentation 2.1 Segmentation (or any general name you specified) 2.1.1. resources 2.1.2. resources 2.2. segmentation - land use & land cover 2.2.1. re 2.2.2. re 2.3.segmentation - vegetation, crops & crop boundaries 2.3.1. re 2.3.2. re

The problem with Segmentation, object detection, and super-resolution techniques is that there are no names for the first sub-lists (2.1, 4.1, and 14.1 ). I think the solution mentioned above can work.

robmarkcole commented 1 year ago

Those first sections can be labelled Segmentation - miscellaneous etc

MuhammedM294 commented 1 year ago

Yes, Agree. So, they should be renumbered with their resources to the last sub-section, right?

  1. Segmentation 2.1. Segmentation - Land use & land cover 2.2. Segmentation - Vegetation, crops & crop boundaries . . . 2.13. Segmentation - miscellaneous
MuhammedM294 commented 1 year ago

@robmarkcole It's ready to be reviewed and merged now.

robmarkcole commented 1 year ago

Thanks @MuhammedM294 !

MuhammedM294 commented 1 year ago

@robmarkcole My pleasure!