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

Add data loader for spam dataset with tests and documentation #1224

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Prateek0xeo
Copy link

This PR adds a new data loader for the spam dataset. The data loader allows users to load the dataset directly into torch for training and testing machine learning models.

Changes Made:

  • Added spam_dataset to torch/datasets.
  • Created unit tests for the loader.
  • Updated documentation and examples to include the spam dataset.

Issues encountered:

  • lantern build process was fail

@sebffischer
Copy link
Collaborator

sebffischer commented Dec 30, 2024

Maybe this PR fits better in https://github.com/mlverse/torchdatasets (I am not the maintainer though, so maybe @dfalbel has a different opinion)

@Prateek0xeo
Copy link
Author

Prateek0xeo commented Jan 2, 2025 via email

@cregouby
Copy link
Collaborator

cregouby commented Jan 2, 2025

Hello @Prateek0xeo,

Many thanks for this contribution !

@sebffischer is right, the perfect place for your P.R. is torchdatasets.
Would you be kind enough to make the same P.R. in torchdatasets/pulls ?

@Prateek0xeo
Copy link
Author

Prateek0xeo commented Jan 2, 2025 via email

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.

3 participants