Skip to content
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

Compile all expressions at once when validating. #268

Merged
merged 6 commits into from
Jul 7, 2023

Conversation

aoltean16
Copy link
Collaborator

No description provided.

@aoltean16 aoltean16 force-pushed the feat/designvalidationv2 branch 5 times, most recently from 16a31fa to 2d8f918 Compare June 19, 2023 11:22
@aoltean16 aoltean16 marked this pull request as ready for review June 19, 2023 12:52
@aoltean16 aoltean16 requested a review from LaviniaC June 19, 2023 12:52
@angelowang
Copy link

angelowang commented Jun 22, 2023

Hi @aoltean16 , will this change improve the performance? I had to make some change like in #260 to make it much faster on first Run(). There are many duplicated file access which is time consuming when we have a lot of CodeActivities. But even after that, the performance is still 10x slower than the VbHostedCompiler before :(

Seems it's only for design time?

@aoltean16
Copy link
Collaborator Author

Hi @aoltean16 , will this change improve the performance? I had to make some change like in #260 to make it much faster on first Run(). There are many duplicated file access which is time consuming when we have a lot of CodeActivities. But even after that, the performance is still 10x slower than the VbHostedCompiler before :(

Seems it's only for design time?

This improves the validation time drastically, when called by ActivityValidationServices.Validate() with ValidationSettings.ForceExpressionsCache = false, so in theory, yea, only design time will benefit from this

@angelowang
Copy link

angelowang commented Jun 25, 2023

in theory, yea, only design time will benefit from this

Thanks @aoltean16 . The execution time also needs this eagerly. We have a case where 21 xaml files takes 30s for the first time running. This is still the time after the MetadataReference caching, the original time could be well over several minutes.

The idea might be similar, like each expression's LambdaExpression could be from a separate CreateExpression_actionHash/codeHash/actionId function.

@aoltean16 aoltean16 merged commit 4385aa7 into develop Jul 7, 2023
2 checks passed
@aoltean16 aoltean16 deleted the feat/designvalidationv2 branch July 7, 2023 14:37
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants