PROLEAD icon indicating copy to clipboard operation
PROLEAD copied to clipboard

PROLEAD terminates without error if certain options in config.set are set incorrectly.

Open FelixUhle opened this issue 1 year ago • 0 comments

Describe the bug The release build of prolead does not throw an error if an incorrect number of expected_outputs or no_of_groups is given. Instead, PROLEAD simply closes after "Read settings file..." is issued.

To Reproduce Steps to reproduce the behavior:

  1. Select a working configuration.
  2. Manipulate it in a way that it contains either
% no_of_groups bug
no_of_groups % number of groups to conduct the test, e.g., fixed vs. fixed, fixed vs. random, etc.
2

64'h$$$$$$$$$$$$$$$$
%64'h$$$$$$$$$$$$$$$$

no_of_outputs
2

[31:0] rd_s0
[31:0] rd_s1

expected_output

32'h$$$$$$$$
32'h$$$$$$$$

, or

% expected_output bug
no_of_groups % number of groups to conduct the test, e.g., fixed vs. fixed, fixed vs. random, etc.
2

64'h$$$$$$$$$$$$$$$$
64'h$$$$$$$$$$$$$$$$

no_of_outputs
2

[31:0] rd_s0
[31:0] rd_s1

expected_output

32'h$$$$$$$$
%32'h$$$$$$$$
  1. Execute PROLEAD

Expected behavior We expect a meaningful error message, which explains what the problem is. In the first given case above the error message should contain information that two inputs must be given if no_of_outputs=2. In the second case the error message should inform the user that two expected_outputs must be given if no_of_outputs=2.

Additional context Since it is already planed to replace the current parser with a more robust one, using a standardized format (see #10), we plan to address this bug by introducing the new parser.

FelixUhle avatar Feb 20 '24 11:02 FelixUhle