Skip to content
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

Terrain Classification #810

Merged
merged 9 commits into from
Jul 20, 2024
Merged

Terrain Classification #810

merged 9 commits into from
Jul 20, 2024

Conversation

MohanRocks999
Copy link
Contributor

@MohanRocks999 MohanRocks999 commented Jun 21, 2024

Pull Request for DL-Simplified 💡

Issue Title : Terrain Recognition

  • Info about the related issue (Aim of the project) :
  • Name: Mohana Krishna
  • GitHub ID: https://github.com/MohanRocks999
  • Email ID: [email protected]
  • Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) I am Contributing in GSSoC 2024

Closes: #636

Add-ons or changes made 📃

Trained a Image Classification using various deep learning algorithms. This include DeepViT (Not just Vision Transformer), EfficientNet B3, DeiT3 (Data efficient image transformer) and YoloV8 that classifies images on 5 different types of terrains - coast, mountain, glacier, forest and desert.

Type of change ☑️

  • [ X] New feature - A Terrain Classification using the respective neural network architecture using pytorch.

How Has This Been Tested? ⚙️

The Model have been evaluated extensively with a validation dataset comprising of 2000 images (400 images per class) and a test dataset of 500 images.

Checklist: ☑️

  • [ X] My code follows the guidelines of this project.
  • [ X] I have performed a self-review of my own code.
  • [X ] I have commented my code, particularly wherever it was hard to understand.
  • I have made corresponding changes to the documentation.
  • [ X] My changes generate no new warnings.
  • [X ] I have added things that prove my fix is effective or that my feature works.
  • [ ]X Any dependent changes have been merged and published in downstream modules.

Copy link

Our team will soon review your PR. Thanks @MohanRocks999 :)

@abhisheks008
Copy link
Owner

What's the issue number?

@MohanRocks999
Copy link
Contributor Author

The issue number is 636

@abhisheks008
Copy link
Owner

Hi @MohanRocks999 remove the extra files and follow the project structure for this project.

@abhisheks008 abhisheks008 added Status: Requested Changes Changes requested. gssoc Girlscript Summer of Code 2024 labels Jul 6, 2024
@MohanRocks999
Copy link
Contributor Author

What files needs to be removed and do you want me to add all the models in one single notebook?

@abhisheks008
Copy link
Owner

It'll be better to have a single notebook file (.ipynb) where all the models and EDAs can be implemented together.

@MohanRocks999
Copy link
Contributor Author

The changes has been as per your request. check it out.

@abhisheks008 abhisheks008 changed the title Pull request for adding Terrain Classification using DeepViT, EfficientNetb3, YoloV8, DeiT3 Terrain Classification Jul 12, 2024
Copy link
Owner

@abhisheks008 abhisheks008 left a comment

Choose a reason for hiding this comment

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

Hi @MohanRocks999 everything looks good to me. Some minor changes are required,

  1. Follow the README template and update the README as per the given points. Here is the template: https://github.com/abhisheks008/DL-Simplified/blob/main/.github/readme_template.md
  2. Insert the resulted images inside the Images folder including the confusion matrix, data visualization and so on.
  3. Remove the .gitignore file.

@MohanRocks999
Copy link
Contributor Author

The requested changes has been done. Please check the changes as soon as you can.

Copy link
Owner

@abhisheks008 abhisheks008 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Approved ✅
@MohanRocks999

@abhisheks008 abhisheks008 added Status: Approved Approved PR by the PA. level2 Level 2 for GSSOC Points Updated and removed Status: Requested Changes Changes requested. labels Jul 20, 2024
@abhisheks008 abhisheks008 merged commit 67e324d into abhisheks008:main Jul 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc Girlscript Summer of Code 2024 level2 Level 2 for GSSOC Points Updated Status: Approved Approved PR by the PA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Project Addition] Terrain Recognition
2 participants