Ep 4 suggestions - Githubissues

carpentries-incubator / twitter-with-twarc

Introduction to Harvesting Twitter Data with Twarc

https://carpentries-incubator.github.io/twitter-with-twarc/

Other

4 stars 4 forks source link

Ep 4 suggestions #21

Closed gnunnelley closed 2 years ago

gnunnelley commented 2 years ago

Under “What’s a lot” move !twarc2 common word counts to separate block
Inconsistent naming conventions: e.g. ucsblib.jsonl vs ucsblib_timeline.jsonl
Maybe move !twarc2 timeline --limit 500 UCSBLibrary > 'source-data/ucsblib.jsonl from where it is to ‘jsonl > csv > wc’ section for better flow.
Is list(kittens_df.columns) misplaced?
Everything else is good

kristi-sara commented 2 years ago

all kittens will be replaced with the ucsblib stuff

kristi-sara commented 2 years ago

can I just make it UCSB? ucsb library only throws out 20 something tweets

jonjab commented 2 years ago

I wouldn't say all the kittens. there is still counts.

kristi-sara commented 2 years ago

all the kittens in episode 4. I thought we agreed on this this morning?

jonjab commented 2 years ago

can I just make it UCSB? ucsb library only throws out 20 something tweets

timeline dude. we are downloading the timeline.

I will clarify in the lesson while standardizing filenames.

kristi-sara commented 2 years ago

jonjab commented 2 years ago

can I just make it UCSB? ucsb library only throws out 20 something tweets

i see what you're saying. for sure we need something better at 'recent stream' where I was trying to do booleans. even the ucsb I saved in the notebook doesn't really do the trick for what we want here.

jonjab commented 2 years ago

I think this one can be closed with ep. 4 in its current state. there's still plenty of fixme's in there.