-
Notifications
You must be signed in to change notification settings - Fork 36
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
what's the deal with the API section in Project 2? #58
Comments
It was in years past. It's the get_json_notebook.ipynb and https://github.com/ireapps/pycar/blob/master/project2/json_to_csv_notebook.ipynb notebooks. But if you think it needs beefing up—or it's too much to squeeze in—we can certainly adjust. See what you think. |
How did it go in past years? Was there enough time to cover everything? |
I think so, but my recollection is hazy, honestly. @esagara should remember better since he tackled those projects the last couple of years. Did we get to the json sections, Eric? |
Poking @esagara. Were the JSON sections covered last year? |
At any rate, I'll prepare to cover JSON in case there is time and the group seems ready for it. I'll take the pulse of the attendees to decide on the spot. |
That's a good plan. We can certainly adjust our speed on the fly if need be.
…On Sun, Feb 24, 2019 at 10:37 AM Roberto Rocha ***@***.***> wrote:
At any rate, I'll prepare to cover JSON in case there is time and the
group seems ready for it. I'll take the pulse of the attendees to decide on
the spot.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#58 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AArkMiOahtYzILnf8TCQligAcytz2oXqks5vQrGpgaJpZM4af4Pp>
.
|
There's some mention of using an API in the README of Project 2 but it looks unfinished. Is this still part of the scraping session?
The text was updated successfully, but these errors were encountered: