convert all times to timestamps

[beanumber]

This would be "cleaner". Would it improve performance? Should they be indexed?

[beanumber]

If so, this would help to clarify #5

[beanumber]

Note that the original data contains FlightDate.

[nicholasjhorton]

Does this affect my installation? Does it impact the pending request at Amherst? If so, can you please reply to my earlier note to Aaron Coburn?

Thanks in advance,

Nick

On Aug 31, 2015, at 2:28 PM, Ben Baumer notifications@github.com wrote:

Note that the original data contains FlightDate.

—
Reply to this email directly or view it on GitHub.

[beanumber]

This doesn't (yet), but the addition of cancelled does. The good news is I fixed the create bug #17.

[beanumber]

See (tidyverse#11)

[beanumber]

Note that hadley has made this change to nycflights13

[beanumber]

This is done.

[beanumber]

So now I am thinking that we do this differently. I am thinking:

• store dep_time and arr_time as POSIXct timestamps
• store dep_delay and arr_delay as integers
• drop year, month, day, sched_dep_time, sched_arr_time, hour, minute, and time_hour

That might actually reduce the file size quite a bit. It would break compatibility with @hadley 's nycflights13, but nycflights13 would still be recoverable from airlines -- it would just require a little extra wrangling.

[beanumber]

@nicholasjhorton what do you think of this idea?

[nicholasjhorton]

Sounds reasonable to me.

Nick

On Jun 9, 2016, at 9:40 AM, Ben Baumer notifications@github.com wrote:

@nicholasjhorton what do you think of this idea?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Nicholas Horton
Professor of Statistics
Department of Mathematics and Statistics, Amherst College
Box 2239, 31 Quadrangle Dr
Amherst, MA 01002-5000
https://www.amherst.edu/people/facstaff/nhorton