and/or between filters

[Prahitha]

The functionality is kind of limited and still needs some work. For example, it assumes that the user will not enter: coverage > 30 AND/OR coverage > 40 AND/OR coverage > 50

I would also like to know if the data structure used here is okay or if I should implement an array with priority like earlier. Any feedback is appreciated. Thank you!

[mglb]

This doesn't seem to work in basic cases:

• (empty/default value) tool is icarus -> full table is visible; removing the empty value and leaving either OR or AND as the default should remove this problem.
• (AND) tool is icarus -> OK
• (OR) tool is icarus -> all tools visible with "no matching records"

"Remove all filters" doesn't remove filter from the table. Only filter fields are removed.

Why did you remove support for relation operators in entries (rows)? It has been useful to e.g. search "coverage <= 0 OR >= 100" AND "tool is X".

Code structure seems to be OK at first glance - lets make it work well first and optionally do some small optimization/refactoring later.

[Prahitha]

Thanks for the feedback, I'll work on those issues. Also, remove all filters works fine for me. It takes a awhile to redraw the table though.

Why did you remove support for relation operators in entries (rows)? It has been useful to e.g. search "coverage <= 0 OR >= 100" AND "tool is X".

@tgorochowik suggested that we have only one filter per row. So we agreed to first implement AND/OR between rows as a global relation and then extend it to AND/OR between every row.

[mglb]

Also, remove all filters works fine for me. It takes a awhile to redraw the table though.

Case that don't work in Chrome and Firefox:

• Select OR
• Add
• Select: tool is icarus
• Apply → "No matching records"
• Remove all filters → still "No matching records"

I did wait a few minutes for results.

[Prahitha]

Ah yes, it doesn't seem to work when there are no matching records.

[github-actions[bot]]

Compared test results

tool	new_failures	new_passes	added	removed	not_affected
Slang	0	0	0	0	4974
Verilator	2	1	0	0	4986
Yosys	0	1	0	0	4973
moore	0	0	0	0	4746
Icarus	0	0	0	0	4655
UhdmYosys	1	1	0	0	4972
Surelog	2	0	0	0	4972
Verible	0	0	0	0	4655
VeribleExtractor	0	0	0	0	4655
sv_parser	0	0	0	0	4746
Sv2v_zachjs	2	0	0	0	4972
tree_sitter_verilog	0	0	0	0	4655
Odin	0	0	0	0	4746
UhdmVerilator	0	0	0	0	4989

Download an archive containing all the details