Too much logs printed. It's at least 10x slower.

Question

Too much logs printed. It's at least 10x slower.

Pcosmin opened this issue 4 years ago · comments

Version: 2.2.32
The profiling process print a lot of useless logs which makes the process very slow. I think you should remove the print from optimus/infer.py:309, where it prints the dtypes:
"print(dtypes)"

With "print(dtypes)" it took a very long time, I waited for over 10 minutes to print all the dtypes, and it didn't finish to print all of them. After 10 minutes I just killed the process.
Without "print(dtypes)" it took 96 seconds for the whole profiling process to be done.

issue-label-bot · Answer 1 · Fri Oct 02 2020 20:59:35 GMT+0800 (China Standard Time)

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.55. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

Argenis Leon · Answer 2 · Sat Oct 03 2020 02:56:42 GMT+0800 (China Standard Time)

Thanks for the report @Pcosmin. Could you send PR?

Cosmin · Answer 3 · Tue Oct 06 2020 21:56:35 GMT+0800 (China Standard Time)

Hi @argenisleon . I sent the PR: #1064 . Please let me know if there are other changes that I should do.

Argenis Leon · Answer 4 · Wed Oct 07 2020 10:18:32 GMT+0800 (China Standard Time)

Thanks @Pcosmin. Merged