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.

Sign up for GitHub

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

Jump to bottom

Update with new features #13

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

Update with new features #13

wants to merge 9 commits into from

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)

@kakaroto
avrdude's -D option will disable the auto erase of flash
c2796db
@kakaroto
Add verify and verbose options to avrdude
7c50dcd
@kakaroto
Add support for serials on windows that are not files but start with COM
7510c07
@kakaroto
Print supply values as well as supply voltages
d364491
@kakaroto
Allow unknown version if git is not installed
30b5b88
@kakaroto
Add configuration, windows support and remove homing.
aed6825 
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.
@kakaroto
Skip controller port if not using its serial number and add a functio…
ba154b6 
…n to get serial number from com port
@kakaroto
Add configuration of type of database to use
e120156 
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
Remove import of psycopg2 from test_process and leave it only in the …
d01371e 
…postgresdb.py
@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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kakaroto @sjkelly