codex-team / hawk.workers

Hawk workers system
7 stars 2 forks source link

[performance] Migrate to source-map-js #223

Open kabachook opened 3 years ago

kabachook commented 3 years ago

https://github.com/7rulnik/source-map-js Seems to have better performance and maintained More context: https://t.me/valya_reads_issue/111

neSpecc commented 3 years ago

@robonetphy Could you try benchmarking this?

robonetphy commented 3 years ago

@neSpecc you can check this repo I have done basic time analysis of the both source-map and source-map-js library and also plot a graph

neSpecc commented 3 years ago

@neSpecc you can check this repo I have done basic time analysis of the both source-map and source-map-js library and also plot a graph

Thanks. Please provide your conclusions and screenshots.

robonetphy commented 3 years ago

@neSpecc Actually I can't conclude because you can check the time difference graph of TimeOf(source-map)-TimeOf(source-map-js) as follow. it's always varying while I am passing constant amount of work. I am using performance.now() for time measurement. Let me know if there is better way. newplot (5) newplot (4) newplot (1)