Rockbox

add sqrt() function based on the algorithm BlueChip told me.
Thanks! Math is beautiful.

fix a LCD display bug
sin,cos,log is coming soon

- Add sin, cos function, delete “Del”,”CE” button

- Now accept “num, operator (+,-,*,/), num, operator, ….. ,
= " input sequence

- Add delete animation, button press animation

- fix some bugs

Hey, excellent work!

All i see is one bug: the last number is chopped off…

So if i input

4 * 4 = 16

all i see is
<blank> * <blank> = 1.

Hope this can get fixed, nice job otherwise..

Also, looks like you’re running out of room on that keypad!
:) you could use ON to swap between basic functions and
scientific ones.

-zakk

I couldn’t repeat the bug you said. Can you try the latest
file? There were several display bugs in the initial release.
There might a bug like this before.

“switch” or “2nd” button is not hard to realize. But the
scientific fucntion is a little hard to code. Sin,cos are the
only scientific ones I have:) I have to wait until I have more
to add “2nd” button.

you could move:

sin, cos, log, tan, x^2, x^3, sqrt, and ^ to scientific
leaving all the memory and basic functions to normal mode.

I’ll try the update on tuesday, but i thought i had the most
recent file…

you could move:

sin, cos, log, tan, x^2, x^3, sqrt, and ^ to scientific
leaving all the memory and basic functions to normal mode.

I’ll try the update on tuesday, but i thought i had the most
recent file…

Still having the problem…

This is on the target by the way.

Zakk

Ok, I got the bug. You must be using the latest daily build,
while I am using the official v2.2. There is a change in
snprintf.c since 04/22 build and thereafter, which is a core
funtion for LCD display generation. I updated my program and
tested on 05/12 build.

Besides, I added the “2nd” button and some other functions.
But only sin, cos, n! are working.

Sounds good! I’ll try it again today, then.

Wow! excellent work.

It also looks like all of the 2nd functions work - is this
correct?

.
.
.
Hey dude,

Glad to see you got everything working - looking sharp :)

Hope you’re okay with this little request:

As you may know, the Rockbox guys refuse point-blank to
credit me for my work - it would be real nice to see my name
credited in your source code.

No great banner, just a “thanks to BlueChip for the sqrt()
function” (or whatever) would be nice.

I realise that only other techies will ever see it in there - but
it’s always nice to feel appreciated

Hang loose,

BlueChip
.
.
.

Zakk:

No, the other 2nd funtions are not working. I forgot to
disable them and they just overlapped with the buttons on
1st screen. I’ve fix this.

I am just thinking: with the help of F1,F2,F3 shortkey, we
probably don’t even need 1st screen.

BlueChip:

Sure, I can add thanks credit. while then I also added the
name of the college teacher who told me the name of the
algorithm. But I recently planed to use the unified method to
do trigonometric, hyperbolic, exponential and logarithmic, as
well as square root.

Now the disinformation starts (again).

“the Rockbox guys” do not refuse credit to anyone. We
include the full name of every known Rockbox contributor we
know of.

You refuse to state your name so we can’t include it. That’s
your choice, not ours.

Badger,

I’m glad to discover I am wrong - it is the second best way to
learn (The best way being to discover that someone else who
shares your views is wrong ;) )

My very real & well known online name is and has always been
“Bluechip” (for some 20-odd years now.) I consider the details
on my Birth Ceritifcate, Passport, Bank Account etc. to be
“private & personal.” I am sure you can appreciate that.

Cheers,

BlueChip

Added to CVS just now!

not sure if bug reporting has to be done here, but i’ll just do it.
the M+ doesn’t always work correct.
Example:
press 8, M+, MR → 8 is in memory
now press CE, M+ so you should add 0 to the memory.
press MR → 0.8 is in memory!
press 18, M+, MR → 18 is in mem
now press CE, M+, and 1.8 is in memory.
now thinking of it, maybe i can fix it myself. But I’m lazy and
too busy right now :)

woo~~ I don’t even know that this patch tracker still exists
after it was added to CVS. I can’t find it through search or
browse. Anyway, thanks for the bug report, I’ll check it for
sure. just don’t know how to change the official file in CVS.

it seems i can search better i think you need cvs acces (which you have to earn by writing
consistently good code, i read) or ask someone with cvs
access to do it (just Daniel Stenberg again, or ask a bit around
on #rockbox, i guess…)
i hope you can find the bug. nice calc btw :)

I found the bug several days after Linus did. Linus has made
change in CVS. The bug is because I forgot a pair of
brackets:)

ah… well, nice that it’s fixed now :)