New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bf-lu perf updates #554
bf-lu perf updates #554
Conversation
…into vishwac/472 m master
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cil
@munozemilio updated. I also noticed that separating the qna from qna alterations causes all files to parsed twice which is expensive. We should probably have qnabuilder do both qnaList as well as qnaAlterations with a single parse. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CIL
Made bunch of fixes to improve perf for core bf-lu operations.
Notable example include large .lu files - e.g. email skill.lu
This multi-file convert now takes ~1,715ms .vs. 5,704ms, better by about 70%
The BF-LU tests now take ~89s .vs. ~129s, better by about 30%.