tuura / scenco-core

Collection of encoding algorithms for conditional graphs
Other
0 stars 1 forks source link

ABC flow (synthesis, mapping) supported #19

Closed allegroCoder closed 8 years ago

allegroCoder commented 8 years ago
snowleopard commented 8 years ago

I added a bunch of comments above :)

The main issue is that for some reason you are changing a perfectly clear strongly typed API with some error-prone and obscure one, where parameters are stored in files? This is even worse than having String parameters everywhere. Please bring the nice API we had back!

allegroCoder commented 8 years ago

@snowleopard now you can have a look at the PR for merging. Please, let me know if you want me to modify anything else. I have modified Formula.hs quite a lot. All the functions in there are useful for loading and unloading formulae into SCENCO for the technology mapping process, none of them are exported to the external environment though (following the approach we discussed if I am not mistaken).