search

USU-Robosub / Poseidon

USU's third-generation autonomous submarine
http://usub.cs.usu.edu
17 stars 0 forks source link

Add IMU Controller #26

Closed nfcopier closed 8 years ago

nfcopier commented 8 years ago

We need to be able to receive data from the IMU.

The I feel like the best implementation is to have the Node interface constantly pulling for data and caching the most recent data set.

*A note on acronyms in code: The generally accepted convention for acronyms in code is to capitalize only the first letter, i.e.: ImuController vs IMUController.

TekuConcept commented 8 years ago

IMU is an acronym in and of its self. It stands for Inertial Measurement Unit; Thus I capitalized it as follows: "IMUController" (In addition, it also looks nicer)

On Jan 7, 2016, at 10:02 PM, Nathan Floris Copier notifications@github.com wrote:

We need to be able to receive data from the IMU.

IMU Controller is initialized with a connection to the Node app. IMU Controller receives data from the drivers. Node app has interface to the IMU Controller. The I feel like the best implementation is to have the Node interface constantly pulling for data and caching the most recent data set.

*A note on acronyms in code: The generally convention for acronyms in code is to capitalize only the first letter IE: ImuController vs IMUController.

― Reply to this email directly or view it on GitHub.

nfcopier commented 8 years ago

I know it's an acronym. That's why I put that note. I would disagree that IMUController looks nicer. I find the lack of clear separation of words makes it difficult to read. This is why most linters discourage that type of capitalization.

TekuConcept commented 8 years ago

The IMU driver code that uses I2C needs to be implemented on the Arduino while the serial interface needs to be done on the Bone. The existing functions such as "sealevelPresure" also should be implemented on the Arduino in order to preserve efficiency with data transfers.

butterfieldj commented 8 years ago

Yep, I will get to work on this hopefully tomorrow. Is anyone planning on being in the lab tomorrow? I still don't have a key card.

On Mon, Feb 1, 2016 at 10:45 AM, TekuConcept notifications@github.com wrote:

The IMU driver code that uses I2C needs to be implemented on the Arduino while the serial interface needs to be done on the Bone. The existing functions such as "sealevelPresure" also should be implemented on the Arduino in order to preserve efficiency with data transfers.

— Reply to this email directly or view it on GitHub https://github.com/USU-Robosub/Poseidon/issues/26#issuecomment-178091858 .

TekuConcept commented 8 years ago

I may be there along with Nathan, and maybe Kevin? Tomorrow we also have the regular Tuesday meeting. (Don't worry about the files: BMP085, HMC5883L, and MPU6050, those are going on the Arduino because they are 'driver' files - I should have them tested by the end of the day)

butterfieldj commented 8 years ago

Ok, what time are you going to be there?

On Mon, Feb 1, 2016 at 1:54 PM, TekuConcept notifications@github.com wrote:

I may be there along with Nathan, and maybe Kevin? Tomorrow we also have the regular Tuesday meeting. (Don't worry about the files: BMP085, HMC5883L, and MPU6050, those are going on the Arduino - I should have them tested by the end of the day)

— Reply to this email directly or view it on GitHub https://github.com/USU-Robosub/Poseidon/issues/26#issuecomment-178184430 .

TekuConcept commented 8 years ago

Maybe before 12pm, but guaranteed after 4:15pm

butterfieldj commented 8 years ago

Sounds good, I'll see you guys there.

On Mon, Feb 1, 2016 at 1:59 PM, TekuConcept notifications@github.com wrote:

Maybe before 12pm, but guaranteed after 4:15pm

— Reply to this email directly or view it on GitHub https://github.com/USU-Robosub/Poseidon/issues/26#issuecomment-178186549 .

TekuConcept commented 8 years ago

Resolved by #93.