ssvlab / dsverifier

DSVerifier - Digital Systems Verifier
http://dsverifier.org
Apache License 2.0
12 stars 4 forks source link

Verification of settling time #117

Closed lennonchaves closed 6 years ago

lennonchaves commented 6 years ago
lennonchaves commented 6 years ago

@lucasccordeiro Could you please start to review the new code for verification of settling time?

lucasccordeiro commented 6 years ago

May I also ask you squash the commits? There are many commits with the same message (e.g., fixing, fixing, updating, updating, etc).

lucasccordeiro commented 6 years ago

You also need to provide regression tests to check the implemented functions.

lennonchaves commented 6 years ago

@thiago-cavalcante, please check the comments

lucasccordeiro commented 6 years ago

In addition to the usual regression tests, you could also provide unit tests to check the implementation of each function.

thiago-cavalcante commented 6 years ago

@lucasccordeiro Do you use any specific tool for unit tests?

lucasccordeiro commented 6 years ago

https://libcheck.github.io/check/ http://aceunit.sourceforge.net/

thiago-cavalcante commented 6 years ago

@lucasccordeiro Could you please start to review the new code for verification of settling time?