Bug 104695 - Problems in the equation parser
Summary: Problems in the equation parser
Status: RESOLVED REMIND
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-04-28 13:24 UTC by Netterfield
Modified: 2005-08-08 21:03 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Netterfield 2005-04-28 13:24:31 UTC
Version:           1.1.0_beta1 (using KDE 3.4.0, compiled sources)
Compiler:          gcc version 3.3.5
OS:                Linux (i686) release 2.6.9

Equation 1 parses correctly and gives the correct number of
samples per frame:

([GYRO3]*cos([EL]*[C_D2R])-[GYRO2]*sin([EL]*[C_D2R]))

Equation 2 only gives 1 sample per frame, even if interpolate is set (note the unary minus):

-([GYRO3]*sin([EL]*[C_D2R])+[GYRO2]*cos([EL]*[C_D2R]))

Equation 3 complains that the operator needs to operands (note the multiply at the end)

([GYRO3]*sin([EL]*[C_D2R])+[GYRO2]*cos([EL]*[C_D2R])) * -1

Equation 4 works fine, including correct # samples/frame

(-1)*([GYRO3]*sin([EL]*[C_D2R])+[GYRO2]*cos([EL]*[C_D2R]))
Comment 1 Netterfield 2005-04-28 14:11:30 UTC
> Equation 3 complains that the operator needs to operands ....


Should be "... needs two operands ..."
Comment 2 George Staikos 2005-04-29 02:01:12 UTC
CVS commit by staikos: 

This is taking too much time to debug, so for now I have to remove some of the
verbose error messages in order to fix the parse error they introduced.
CCBUG: 104695


  M +259 -365  eparse.c   1.18
  M +2 -36     eparse.y   1.18
Comment 3 George Staikos 2005-04-29 02:04:34 UTC
  Can you explain the samples per frame problem better?  Perhaps a testcase?
Comment 4 George Staikos 2005-06-16 18:24:04 UTC
Barth, any update here?
Comment 5 George Staikos 2005-08-08 21:03:47 UTC
Need to know what else is wrong here, if anything.