Starter Project for Directed Research in Robotics - Githubissues

rlworkgroup / dowel

A little logger for machine learning research

MIT License

32 stars 37 forks source link

Starter Project for Directed Research in Robotics #50

Closed late-in-autumn closed 4 years ago

late-in-autumn commented 4 years ago

This attempts to allow a more robust handling of dynamic fileds in module dowel. This is implemented by the following logic:

The CSV headers are now ordered
When a new field is encountered, it is appended to the ordered list of headers
The output files gets flushed and re-read to allow header replacement
The new records with new fields get written

In order to accompanish this, these following changes to internal data structures and logic are added:

csv._fieldnames becomes list() instead of set()
The output file is now opened in w+ mode to allow both read and write access

Unit tests were added for the following cases:

Adding a new field at the end of the list of fields
Adding a new field in the middle of the list of fields

Sincerely

late-in-autumn commented 4 years ago

Oops, wrong branching. Closing the PR.