Skip to content

Update citation for plant_village dataset#4137

Open
spMohanty wants to merge 1 commit intotensorflow:masterfrom
spMohanty:master
Open

Update citation for plant_village dataset#4137
spMohanty wants to merge 1 commit intotensorflow:masterfrom
spMohanty:master

Conversation

@spMohanty
Copy link
Copy Markdown

Updating the citations for the dataset source, as this dataset in its current form was released in this paper via this github repository by the same authors who made the initial arXiv release (http://arxiv.org/abs/1511.08060).

Thank you for your contribution!

Please read https://www.tensorflow.org/datasets/contribute#pr_checklist to make sure your PR follows the guidelines.

Add Dataset

  • Dataset Name:
  • Issue Reference:
  • dataset_info.json Gist:

Description

Checklist

  • Address all TODO's
  • Add alphabetized import to subdirectory's __init__.py
  • Run download_and_prepare successfully
  • Add checksums file
  • Properly cite in BibTeX format
  • Add passing test(s)
  • Add test data
  • If using additional dependencies (e.g. scipy), use lazy_imports (if applicable)
  • Add data generation script (if applicable)
  • Lint code

Updating the citations for the dataset source, as this dataset in its current form was released in [this paper](https://www.frontiersin.org/articles/10.3389/fpls.2016.01419/full ) via [this github repository](https://github.com/spMohanty/PlantVillage-Dataset) by the same authors who made the initial arXiv release (http://arxiv.org/abs/1511.08060).
@ccl-core ccl-core added copybara-import Internal label for PR management and removed copybara-import Internal label for PR management labels Sep 15, 2022
"""

_URL = "https://data.mendeley.com/public-files/datasets/tywbtsjrjv/files/d5652a28-c1d8-4b76-97f3-72fb80f94efc/file_downloaded"
_URL = "https://github.com/spMohanty/PlantVillage-Dataset"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @spMohanty, thanks a lot for the citation update!

It seems the _URL variable is used a few lines below to download and extract the dataset. So changing it might be breaking. You can try to rebuild the dataset by running tfds build to see if it breaks anything.

Why do you need to change the URL?

Copy link
Copy Markdown
Collaborator

@OmkarBorhade98 OmkarBorhade98 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants