search

SacArea-BayDeltaProjects / HydroRDITemplate

CR1000 and CR6 Datalogger template for RDI ADCPs
2 stars 0 forks source link

Overhaul of Pubilc/Dim Variables #57

Open violetteta8 opened 5 years ago

violetteta8 commented 5 years ago

I'm going to create a new branch to work on overhauling the variables/subroutines and the conditionally compiled stuff. I'd like to get everything set appropriately and eliminate unwanted variables depending on the conditionals, etc. I think this includes arranging subroutines in a logical order as well, that way searching for bugs will be easier. This will be a process and I think would be considered V 5.xx.

I'm going to start today, but I'd like input as well. I think this overhaul could be good for us to work on together.

violetteta8 commented 5 years ago

I'll make a branch called "Overhaul" and then I'll branch off of that so we can have pull requests to the Overhaul branch so we don't step on each others' toes.

I think it makes sense to essentially eliminate the "Debugging" conditional and keep everything in the Public table now that we have the SVMaq Table.

violetteta8 commented 5 years ago

All branches were deleted after the SUNA and Iridium additions. So, start a fresh branch off the master for this issue.