search

openworm / tracker-commons

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

Added support for scalar multipliers on units to Scala. #75

Closed Ichoran closed 8 years ago

Ichoran commented 8 years ago

Also added a description of the feature to the long explanation of units.

MichaelCurrie commented 8 years ago

Very minor but would it be better to call TestWcon.scala, TestWCON.scala?

Ichoran commented 8 years ago

I'm not sure. There's certainly no shortage of "Json" instead of "JSON" in names of JSON parsers for Scala. For example, the most commonly used JSON parser is part of the Play framework, and look how they capitalize things.

So I think it's sorta standard for the language to not necessarily follow strict capitalization for acronyms, and instead use CamelCase. If it is too jarring with the rest of the tracker-commons software, I don't think it would be a problem to change it, though.

MichaelCurrie commented 8 years ago

Up to you. I don't think it's jarring one way or the other.