CSV log writer breaks on some platforms

apprenticelearner / AL_Train

A repository for the CTAT HTML based training harness for Apprentice Learner agents.

MIT License

5 stars 5 forks source link

CSV log writer breaks on some platforms #3

Open cmaclell opened 5 years ago

cmaclell commented 5 years ago

We need to set a default utf-8 encoding to instances of the CSV log writer in AL_HTML because some international platforms will default to an incompatible setting.

eharpste commented 5 years ago

It would be great if we had a place to specify this as a configuration parameter rather than locking everything into UTF-8 with in-line parameters. Seems like the best place to do that here would be in net.conf, as the only configuration file for this project, but that seems like the wrong place generally.

eharpste commented 4 years ago

This was hot fixed as part of the summer school last year and may not have been fully caught. We should double check this is implemented everywhere we write a file.