Skip to content
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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

kakaroto
Copy link
Contributor

I'm sending this update with a few new features for the RAMBo Uploader. Most noteable new features are :

  • Added windows support
  • Automatic detection of serial number (only works on windows for now)
  • separate configuration file to control the test process
  • Use of the serial number to detect the controller and to ignore boards (if you have a printer connected to the same PC)
  • Use of avrdude class to program/verify the ICSP (configurable)
  • Added log file support instead of database (configurable)
  • Removed homing (not needed for new jig design)

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.
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)
@kakaroto
Copy link
Contributor Author

Hi,
Any plans to review/merge these changes ? I'd like to get those out of the way before I start refactoring and integrating the test process with the GUI.
Thanks

@sjkelly
Copy link
Contributor

sjkelly commented Feb 28, 2014

Hi Kakaroto,

First, sorry for the delay.
I've been going through the changes, and they look good so far. I will comment on anything we should discuss. I am not opposed to merging them, since AFAIK UltiMachine is running the current master with some tweaks. I'd like to hear what @johnnyr and @tonokip think first.

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

@sjkelly
Copy link
Contributor

sjkelly commented Jul 17, 2014

@johnnyr is the PC at UltiMachine on a branch? Could we merge this into master since it looks good to go?

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.

2 participants