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.

Sign up for GitHub

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

Jump to bottom

to_numpy method for text dataset #60

Merged
merged 1 commit into from
Apr 9, 2021
Merged

to_numpy method for text dataset #60

merged 1 commit into from
Apr 9, 2021

Conversation

sahilgupta2105
Copy link
Collaborator

[#13 ]

Put an x inside the applicable checkboxes [ ].

Primary type of change:

  • New feature.
  • Bug fix.
  • Refactor.
  • Documentation.
  • Build/ devops.
  • Test.
  • Other.

Checklist:

  • I have actually ran this code locally to make sure it works.
  • I have ran the existing tests, and inspected areas that my code touches.
  • I have updated the tests to include my changes.
  • I have included updates to the documentation.
  • I am pushing a branch, not my main/ master.

Contains breaking changes:

  • Yes (explain what functionality it breaks).
  • No

@sahilgupta2105 sahilgupta2105 linked an issue Apr 9, 2021 that may be closed by this pull request
Ingest text data via Dataset.Text #13
Closed
@sahilgupta2105 sahilgupta2105 self-assigned this Apr 9, 2021
@sahilgupta2105 sahilgupta2105 requested a review from aiqc April 9, 2021 21:38
@sahilgupta2105
Copy link
Collaborator Author

trying to close the loop before moving onto a different issue

@aiqc aiqc merged commit f654e23 into aiqc:main Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aiqc aiqc aiqc approved these changes

Assignees

@sahilgupta2105 sahilgupta2105

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Ingest text data via Dataset.Text
2 participants
@sahilgupta2105 @aiqc