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

script for covid19_italy and titanic dataset from kaggle #60

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

script for covid19_italy and titanic dataset from kaggle #60

wants to merge 10 commits into from
+323 −31

Conversation

DumbMachine
Copy link
Contributor

This is a test script for testing the feature from #1426.

henrykironde
henrykironde reviewed Feb 25, 2020
View reviewed changes
scripts/titanic.json Outdated Show resolved Hide resolved
scripts/titanic.json Outdated Show resolved Hide resolved
scripts/titanic.json Outdated Show resolved Hide resolved
@DumbMachine DumbMachine changed the title script for iris dataset from kaggle added script for iris and titanic dataset from kaggle added Feb 25, 2020
@henrykironde
Copy link
Contributor

We need to remove the iris_kaggle script because we already have iris in the retriever. Look for some other data.

@DumbMachine
Copy link
Contributor Author

DumbMachine commented Mar 18, 2020 •
edited
Loading

I am working on the requests changes, as a substitute of iris_kaggle script i will add a script for COVID-19 data from italy

@henrykironde
Copy link
Contributor

We shall test and merge the scripts after the kaggle PR goes in to avoid any failures

@DumbMachine DumbMachine changed the title script for iris and titanic dataset from kaggle added script for covid19_italy and titanic dataset from kaggle Mar 18, 2020
@henrykironde henrykironde mentioned this pull request Mar 21, 2020
Closed
Kadam-Tushar added a commit to Kadam-Tushar/retriever-recipes that referenced this pull request Jan 26, 2021
@Kadam-Tushar
Fixed kaggle scripts from weecology#65-68 and weecology#60
7afb3c2
@Kadam-Tushar Kadam-Tushar mentioned this pull request Jan 26, 2021
Closed
Kadam-Tushar pushed a commit to Kadam-Tushar/retriever-recipes that referenced this pull request Feb 7, 2021
@DumbMachine @Kadam-Tushar
Fixed kaggle scripts from weecology#60
93dfba8
@Kadam-Tushar Kadam-Tushar mentioned this pull request Feb 7, 2021
Merged
henrykironde added a commit that referenced this pull request Feb 9, 2021
@henrykironde
Merge pull request #91 from Kadam-Tushar/DumbMachine-kaggle-scripts
f04919b 
Fixed kaggle scripts from #60
Base automatically changed from master to main February 11, 2021 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henrykironde henrykironde henrykironde left review comments

Assignees
No one assigned
Labels
Awaiting Final Changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DumbMachine @henrykironde