cpc / openasip

Open Application-Specific Instruction Set processor tools (OpenASIP)
http://openasip.org
Other
138 stars 41 forks source link

explorer: Fix "--pareto_set" if more than 1 app is in the Database #220

Closed nrother closed 1 year ago

nrother commented 1 year ago

Currently DSDBManager::paretoSetConnectivityAndCycles() is always called with the default argument of "-1", meaning that all applications should be taken in to account. This would trigger an assert if there was more than one in the database.

This commit changes the behavior to use the average cycle count in this case. This metric is also used in other places like ConnectionSweeper.

Also convert the pareto_viz script to Python 3.