CurieIMU scoping of methods makes sub-classing difficult. about arduinocore-arc32 HOT 5 CLOSED

@SidLeung, can you see any issues with the above reg_* function delclarations being moved into "public" so users can access them?

from arduinocore-arc32.

Understood the request for the accessing the BMI internals. Please be aware, from our POV, the arrangement/decision of wrapping and hiding the BMI class/info behind the CurieIMU class. In a future event, if it ever occurred, that the BMI160 is replaced, the significant changes would be limited within the boundary of the CurieIMU class. The existing Sketches that utilize the CurieIMU would not be impacted. It is not desirable for us to expose the BMI class/methods for this particular reason.

from arduinocore-arc32.

I agree with @SidLeung.

from arduinocore-arc32.

It is not desirable for us to expose the BMI class/methods

How meaningful is CurieIMU's abstraction, really? Last time I looked, you're exposing quite a lot of BMI specific implementation, like the raw integers where you leave scaling to the end user, and you require the end user to know the specific ranges the BMI sensor implements. A long list of functions correspond 1:1 to BMI's specific motion processing features, which are similar but not identical to other chips on the market.

Look, I don't want to be too negative and cynical here. But if you really in your heart believe some measure of hardware independence is a good thing, perhaps you ought to take an honest objective look at the API you're currently publishing. It leaks a lot of BMI hardware specific details.

from arduinocore-arc32.

I'm not publishing an API. In my case, knowing the details of the hardware (BMI160) was critical to/preferable for the application. I was simply suggesting that it wouldn't take much effort to allow someone to bypass the HAL, especially if the intent was to play around with the Arduino101 they have in hand (like me). At the time I was working on this, the BMI160 datasheet was the best source of information on the Curie.

from arduinocore-arc32.