parallaxsw / OpenSTA

GNU General Public License v3.0
52 stars 23 forks source link

Support JSON format and redirect to file for `report_units` #50

Closed akashlevy closed 4 months ago

akashlevy commented 4 months ago
akashlevy commented 4 months ago

@jjcherry56 let me know if you have interest in upstreaming this. If so, I can add it to the doc as well

jjcherry56 commented 4 months ago

I can't imagine why it would be useful to print json to the console instead of a file, so it only makes sense when used with redirection. I don't know the context of why json for units is useful, but suggest defining a tcl proc sta::write_unit_json filename instead.

akashlevy commented 4 months ago

Yes, this is only useful when other tools need to ingest the units for post-processing. I can move this to a separate tcl proc.

akashlevy commented 4 months ago

Just gonna make a new PR