Split CSV lines in at most 2 places #17
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously, if a transition label contained a comma, the CSV loader would truncate the label there.
This causes the following lines to be considered as transitions with the same label of
"s4(d1"
:The above transitions are found in the
cwi_1_2.csv
benchmark file. This issue caused only 9 enabledness-classes to be detected instead of 11.This fix changes the behavior so that a line is split in at most 3 parts, everything after the second comma goes into the label.