Closed jzhou1318 closed 2 months ago
Hey Jennifer,
Thanks for the user feedback.
I have added you to the HISIM development repo that we internally use. Please commit your changes there in the future for easier integration. I will integrate the current changes from your forked location this time.
I will clarify the PPA.csv queries in the issue #4 response.
Regards, Pragnya
I've been trying to integrate HiSim with a design space explorer ArchGym and have some user feedback.
I've created a wrapper script (copy-pasted below) that I think would be useful for others and would love to see it up-streamed. The script:
analy_model.py
file. This is necessary for external tools to best interact with HiSimset_num_pe
. This is useful when doing design space exploration. Combined with the new way to generate PPA.csv, all results are congregated in one spot for easy analysis. A small example is left at the end of the script.PPA.csv confusion
results_list
correspond with what metric and the units of the metric. Would appreciate more comments in the code next to eachresults_list.append()
call.compute_imc
is printed but logged.Change relative to absolute paths for more streamlined integration with other codebases. I changed
aib_2_5d.py
andutil_mapping.py
in my local version.Change system exits to throwing exceptions. Ending the entire program during design space exploration when encountering a bad configuration is unideal. I'm working on implementing this in my local version (twice in
util_mapping.py
and once inanaly_model.py
, planning to just return a list with a single entry as the error string)Edit: I've forked version 1 of the repo and implemented many changes there https://github.com/jzhou1318/HISIM