matiasandina / FEDWatcher

Software and Hardware to connect FED3 devices over serial on Raspberry Pi 4
MIT License
4 stars 1 forks source link

Column names are a mess and might create parsing errors down the line #42

Open matiasandina opened 1 year ago

matiasandina commented 1 year ago

This is how we define them, but they might vary depending on the library version and the way we construct the string using sprintf

https://github.com/matiasandina/FEDWatcher/blob/0aa62f525c8837e54b73367f57dca37c6986420a/fedwatcher/src/fedwatcher.py#LL42C5-L42C5

We also have made some decisions, for example interPelletIntervalStr gets passed as InterPellet_Retrieval_Time to fedwatcher. We changed the name to avoid overwriting the original (which will have Timed_out instead of the proper value)