-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update with new features #13
base: master
Are you sure you want to change the base?
Conversation
Added a separate configuration file to ease config of app, and added support for windows by having auto-detection of available COM ports and use of serial numbers for controller and target. With new Test jig that doesn't require homing, I also removed the homing state from the code.
…n to get serial number from com port
Added results to log file, no database and postgres db. Also, the serial number is now automatically fetched from USB (works only on windows for now, need to find an alternative for linux)
Hi, |
Hi Kakaroto, First, sorry for the delay. Your changes will help develop a more coherent testing ecosystem. I believe we should work off of them. I'd like to add unit tests, which I think is some low hanging fruit I can pick without being on location. Maybe I should write the unit tests before merging your code? Steve |
@johnnyr is the PC at UltiMachine on a branch? Could we merge this into master since it looks good to go? |
I'm sending this update with a few new features for the RAMBo Uploader. Most noteable new features are :