Opened 9 years ago

Closed 9 years ago

#4807 closed enhancement (fixed)

Improving the Calculate Activity

Reported by: gauravm043 Owned by:
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Calculate Version: Unspecified
Severity: Unspecified Keywords: Calculate Activity
Cc: Distribution/OS: Ubuntu
Bug Status: Resolved

Description (last modified by gauravm043)

I tried some erroneous statements like plot(10) ..
(Undefined) was reported which is good.
But on doing log(-10),asin(10),cot(0) etc. No useful information was given. One feature can be to provide knowledgeable statements like "log of negative number does not exist".

More specifically there is no error class for catching the argument error exception.

Attachments (2)

Changes.diff (3.3 KB) - added by gauravm043 9 years ago.
Hi , I have rectified the errors mentioned in the ticket along with correcting ln(x), inv(x) . godiard and gonzalo_ Please review.
Changes.2.diff (3.3 KB) - added by gauravm043 9 years ago.
Hi , I have rectified the errors mentioned in the ticket along with correcting ln(x), inv(x) according to the latest repository upload . godiard and gonzalo_ Please review.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by gauravm043

I am working on improvising this activity, however any suggestions on how to proceed or some coding styles that i should follow are welcomed :) .

comment:2 Changed 9 years ago by gauravm043

I am working on improvising this activity.

comment:3 Changed 9 years ago by godiard

Would be good create a list of conditions and messages to show.
Another thing would be improve the way error messages are displayed.

comment:4 Changed 9 years ago by godiard

  • Component changed from Sugar to Calculate

comment:5 Changed 9 years ago by godiard

  • Summary changed from Improvising the Calculate Activity to Improving the Calculate Activity

comment:6 Changed 9 years ago by gauravm043

godiard Thanks for your suggestions I have one question though do i have to fork the latest source code from http://git.sugarlabs.org/projects/calculate or is there any other link ?

Changed 9 years ago by gauravm043

Hi , I have rectified the errors mentioned in the ticket along with correcting ln(x), inv(x) . godiard and gonzalo_ Please review.

Changed 9 years ago by gauravm043

Hi , I have rectified the errors mentioned in the ticket along with correcting ln(x), inv(x) according to the latest repository upload . godiard and gonzalo_ Please review.

comment:7 Changed 9 years ago by gauravm043

  • Bug Status changed from Unconfirmed to Resolved
  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.