oss-slu / Mouser

GNU General Public License v3.0
1 stars 7 forks source link

OS patch #217

Closed NNewbauer closed 2 months ago

NNewbauer commented 2 months ago

Capstone Week 1 Changes

OS Issues

  1. A reference to the issue that the PR resolves.

    • Issue is not from list of issues – However, tech lead approved on issue
    • Fixed OS compatibility; Now works on MacOS and Linux
  2. What was changed: a brief summary of the changes you made. Please note that you should not be listing files that you changed or code that you wrote here. This is a summary of what the end result of your changes are, rather than the code itself.

  1. Why was it changed: a justification for why those changes were needed

    • Changed because I was not unable to run Mouser at all.
    • Mouser was Windows specific despite what the ReadMe file said.
    • Since we don't use an RFID controller, we needed a way to access the experimentation page without the use of a controller.
  2. How was it changed: a more technical description of the changes that were made. This is additional information that your teammates will be using when reviewing your code changes, so it's a great place to share some of your thoughts about how you changed the code.

Summary

Changes were made to make program run on Unix systems, also minor debugs to run without a controller plugged in. Changes for OS were made mainly through file path conditionals. Program could not properly run without a controller, in order to get to simulated RFIDs I had to write a conditional that would skip the controller input in order to get to data gathering menu.

jackbelyeu commented 2 months ago

Can you submit a new PR or edit this one to remove the large .env folder?

NNewbauer commented 2 months ago

Can you submit a new PR or edit this one to remove the large .env folder?

It should be gone now

jackbelyeu commented 2 months ago

Related to Issue #220