Skip to content

Conversation

@yaosarayin
Copy link

No description provided.

@wcpettus
Copy link

wcpettus commented Oct 9, 2025

will need to clean up the PR before merging

  • there are two pid_loop files - pid_loop.py and pid_loop_v2.py - are both relevant or is one superseded?
  • there's a tester directory, is that needed still?
  • logs directory probably to be purged wholly (csv data files, plotter script and png files)
  • ipynb files?
    (recognizing that I haven't reviewed the files yet, but will help focus review by cleaning to fewer files)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants