search

openworm / tracker-commons

Compilation of information and code bases related to open-source trackers for C. elegans
11 stars 12 forks source link

kilo prefix #115

Closed JimHokanson closed 8 years ago

JimHokanson commented 8 years ago

In the format where it discusses units, K is allowed for the kilo prefix. Why is this? I'd prefer it not to be ...

MichaelCurrie commented 8 years ago

I agree @JimHokanson; according to SI:

It has been used in the International System of Units where it has the unit symbol k, in lower case.

Happily, the Python implementation already only allows only k and not K for kilo.

It looks like we should change the spec (the schema does not need to be changed as it does not get into this much details).

Ichoran commented 8 years ago

The Scala implementation also only admits k. I agree that it's just an error in the language of the specification--let's change it.

MichaelCurrie commented 8 years ago

Normally I would do a pull request but this was a two-character change so I just committed directly to master: 8cbfd4bb1ef8e572cec336a6316599cb41af6696