Skip to content

Update with new features#13

Open
kakaroto wants to merge 9 commits intoultimachine:masterfrom
kakaroto:master
Open

Update with new features#13
kakaroto wants to merge 9 commits intoultimachine:masterfrom
kakaroto:master

Conversation

@kakaroto
Copy link
Contributor

@kakaroto kakaroto commented Nov 27, 2013

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

kakaroto commented Feb 19, 2014

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