StoEx parsing issue for doubles when simulating media store

Description

There seems to be an issue with StoEx parsing when simulating MediaStore. I chose to run the launch config "ms base all 1k" but I assume other are affected as well.

The reported error is as follows:

Environment

  • Latest nightly build / drop

  • MediaStore from SVN

Activity

Show:
Stephan Seifermann
January 25, 2021, 8:30 PM

Fix has been merged into master.

Stephan Seifermann
January 25, 2021, 8:12 PM

The issue stems from a change in the grammar compared to the old grammar. In the technical report (see PDF, page 30) it was valid to use integer tokens in samples for doubles (because integers are also doubles). For instance, DoublePMF[(1;0.5)(2;0.5)(3;0)] was valid (note the integer tokens in the value and the probability). In the new grammar, a double token is enforced.

I suggest to replace all usages of the DOUBLE token with NUMBER, which can be either an integer or a double. If some form of validation is required (which was not in place in the old and the new grammar), this should be done via the validation framework of Xtext instead of trying to enforce rules by the parser.

Fixed

Assignee

Stephan Seifermann

Reporter

Stephan Seifermann

Labels

None

Requirement Category

None

External issue URL

None

Severity

None

Priority

None

Status Whiteboard

None

External issue ID

None

Components

Fix versions

Affects versions

Priority

Critical