Closed taneliang closed 4 years ago
This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: https://vercel.com/mlh-fellowship/scheduling-profiler-prototype/i4j06hzrd ✅ Preview: https://scheduling-profiler-prototype-git-0f8a7-2.mlh-fellowship.vercel.app
Stack PR by STACK ATTACK:
89 Fix lint failure from #88
90 [Optimize flamechart][1/4] Add flamechart to ReactProfilerData
92 [Optimize flamechart][3/4] Rename flame graph -> flame chart
93 [Optimize flamechart][4/4] Split FlamechartView into row views
Summary
Lays groundwork for #75 by reducing the flamechart data passed around the app.
Was also intended to help #50 by moving some division operations from the hot render loops into preprocessFlamechart, but any performance gains are negligible.
Test Plan
yarn flow
: no errors in modified codeyarn lint
yarn start
: no performance differenceyarn test
: tests updated and passing