openlane2 icon indicating copy to clipboard operation
openlane2 copied to clipboard

Add different warning severity levels

Open donn opened this issue 1 year ago • 2 comments

Description

Currently, all warnings are printed without mention of severity. The warnings all just blend in and are ignored as noise. We did try to mitigate that here: https://github.com/The-OpenROAD-Project/OpenLane/pull/2066 But it appears it wasn't enough.

Proposal

We should:

  • Have a list of warnings in the documentation @kareefardi
  • Categorize the warnings by severity (somehow) @mo-hosni

donn avatar Feb 29 '24 14:02 donn

If a warning is severe enough it should be an error. Otherwise categorizing the warnings is too subjective and severity vary depending on the design. Each tool used in the steps of OpenLane can spit numerous warnings and similarly OpenLane itself. In short, I don't think it is a good idea to categorize the severity of warnings. Having said that, we can have different categories but not necessarily indicate severity.

kareefardi avatar Feb 29 '24 14:02 kareefardi

Let's discuss on our Monday meeting - I'm not the one who prompted this 🙂

donn avatar Feb 29 '24 18:02 donn