View Single Post
  #3   Spotlight this post!  
Unread 23-01-2016, 19:04
Andrew Schreiber Andrew Schreiber is offline
Joining the 900 Meme Team
FRC #0079
 
Join Date: Jan 2005
Rookie Year: 2000
Location: Misplaced Michigander
Posts: 4,062
Andrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond reputeAndrew Schreiber has a reputation beyond repute
Re: Simple Motor Calculator for Linux/OS X

Quote:
Originally Posted by SoftwareBug2.0 View Post
You may want to document your build dependencies. On Ubuntu 15.10 I installed go-bindata and golang-go.

To build I had to change "motors.csv" in the makefile to ".". At least the version of go-bitdata that I have only expects to see directories in that position.

The clean target of the makefile can sometimes fail. Consider adding the "-f" flag to "rm".

When the program is run with no arguments it prints an out of range exception rather than an error message that says what it expected. Similarly, there is no "--help".

When an unknown motor name is encountered the output is lots of NaN rather than an error that the program doesn't know about the motor. For example, try typing "cim" rather than "CIM".


Yup, it's on my todo list (pull requests are also accepted)

So --help and some warnings for not supplying enough args, and throw an error on incorrect motor names.
__________________




.