ellipse and circle implementation

[OmarAli3]

@per1234 Can you please tell me what the exact output for the above seketch is needed to be ?

[per1234]

@OmarAli3 I would expect something that looks approximately like this:

XX******XXX
X*OOOOOO*XX
X*OOOOOO*XX
*OOOOOOOO*X
*OOOOOOOO*X
*OOOOOOOO*X
*OOOOOOOO*X
X*OOOOOO*XX
X*OOOOOO*XX
XX******XXX
XXXXXXXXXXX

I might not have the shape of the circle quite right, but your code seems to do that well enough already.

What I expect is a circle that:

• Is 10 characters wide by 10 characters high
• Has its inside filled with Os.
• Doesn't have an extra "pixel" on the right edge.

[OmarAli3]

After working on the problem :

• I have found that the problem of my code is that I calculate the exact "pixel" position in float then it's casted to integer to pass into set() that take only integers, so I suggest to make override set() that take floats and then whatever graphic display library that inherit this library should floor or ceil the "pixel" position in the way that fit its "pixel" size.
• I tested my code in c++ with Borland Graphics Interface library and found that it is working very well as the "pixel" size is very samll and also noticed that as the width and height increase the error disappear.

So now I have two choices:

• To work on the fill problem only and I guess that the "pixel" position problem will be fixed with real graphic diplay devices.
• To find a different approach that in some way it approximate the position values as integers.

@per1234 , Do You suggest something?

[per1234]

I guess that the "pixel" position problem will be fixed with real graphic diplay devices.

As far as I know, this library is currently only used by the Arduino_MKRRGB library (though I'm sure it will be used for many other applications soon). The Arduino MKR RGB shield is a 12x7 LED matrix, so it is actually less of a "real graphic display" in some ways than the Serial Monitor.

Do You suggest something?

Because of the use with the MKR RGB Shield, I do think it's important to avoid this extra pixel. That will be very noticeable when it's an extra LED lit up. I have confirmed the issue also occurs with using the Arduino_MKRRGB library and the Arduino MKR RGB shield.

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[leonardocavagnis]

Fixed by #30