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

Imrooved the data loading process by downloading the data directly from kaggle using API token and Added the documentation for specifying the objective of this project #374

Closed
wants to merge 1 commit into from

Conversation

yuvraaj2002
Copy link

…Imrooved the data loading process by downloading the data directly from kaggle using API token

Pull Request for DL-Simplified 💡

Issue Title : Data loading process improvements for artwork image recognition

  • Info about the related issue (Aim of the project) : Artwork image recognition
  • Name: Yuvraj Singh
  • GitHub ID: yuvraaj2002
  • Email ID: [email protected]
  • Idenitfy yourself: OSEN (Open Souce Enthusiast Network) participant

Closes: #issue number that will be closed through this PR

Describe the add-ons or changes you've made 📃

The following add-ons and improvements have been done in this project

  1. Added a new section to the documentation on the benefits of building the artwork image recognition project, which highlights the following two key advantages ⬇️
  • Automated interpretation of artwork images without the need for human intervention
  • More efficient cataloging and search of artwork collections
  1. Automated the data loading process : The following changes have been made to automate the data loading process and address the corrupt link to the dataset on Kaggle ⬇️
  • The dataset link has been updated with a new working link.
  • A brief description of the two approaches for downloading and loading the dataset into Colab or Jupyter Notebook has been provided.
  • Code has been added to automate the data downloading and loading process from Kaggle using the Kaggle API access token. This code simply requires the user to have their own Kaggle API token and run the cells to automatically download the dataset as a ZIP folder.
    image

Type of change ☑️

What sort of change have you made:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, local variables)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested? ⚙️

Regarding the changes I have made the automated data loading has been tested by myself in both google colab and jupyter notebook.

Checklist: ☑️

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

…Imrooved the data loading process by downloading the data directly from kaggle using API token
@github-actions
Copy link

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

@yuvraaj2002 yuvraaj2002 changed the title Added the docmentation for specifying the objective of this project, … Imrooved the data loading process by downloading the data directly from kaggle using API token and Added the documentation for specifying the objective of this project Oct 25, 2023
@abhisheks008
Copy link
Owner

This project repository is exclusively dedicated to Deep Learning projects. I appreciate your interest, but it may not be the most suitable place for your contributions.
@yuvraaj2002

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

Successfully merging this pull request may close these issues.

2 participants