search

readingschool-cs-club / JayI

A simple AI that talks to you and learns from you as you talk to it.
4 stars 1 forks source link

add csv support to JayI #11

Closed ghost closed 9 years ago

ghost commented 9 years ago

The old files are incompatible but the default filename is different so it shouldn't crash

raehik commented 9 years ago

Ah, something I wanted to implement while trying to refactor too much at once -- why don't we use a .db extension? It's more abstract than .txt or .csv and allows us to arbitrarily switch between formats.

ghost commented 9 years ago

No because it would break when we upgraded the file format!

the old files are incompatible but the default filename is different so it shouldn't crash!

raehik commented 9 years ago

That's the only thing I was thinking about. My main reasoning was that users should not be told the file format unless they need it, but I never made up my mind about whether users should be able to view the database. With CSV it would be useful, because then people could know "oh, I could import this to [Excel]' or w/e.