Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[REF-1682][REF-1683][REF-1684][REF-2283]Benchmark reflex package size and .web folder #2880

Merged
merged 52 commits into from
Mar 25, 2024

Conversation

ElijahAhianyo
Copy link
Contributor

@ElijahAhianyo ElijahAhianyo commented Mar 19, 2024

This Pr we benchmark the following:

  • Reflex build size: We achieve this by running poetry build on the PR and measuring the size of the generated dist folder
  • Reflex install size: This measures the total size of installing reflex(pip install reflex) with all of the dependencies it installs. we achieve this by installing reflex in venv and then calculating the size of the site-packages directory.
  • .web folder size of a base app: We measure the size of the .web folder after running the counter app.
  • .web folder size of reflex web: We also measure the size of the .web folder after running reflex-web.

We then publish the data obtained to a postgres db.
Note that the sizes calculated are published/stored as megabytes.

@ElijahAhianyo ElijahAhianyo changed the title Benchmark reflex package size after install Benchmark reflex package size and .web folder Mar 20, 2024
@ElijahAhianyo ElijahAhianyo changed the title Benchmark reflex package size and .web folder [REF-1684][REF-1683]Benchmark reflex package size and .web folder Mar 20, 2024
@ElijahAhianyo ElijahAhianyo changed the title [REF-1684][REF-1683]Benchmark reflex package size and .web folder [REF-1682][REF-1683][REF-1684][REF-2283]Benchmark reflex package size and .web folder Mar 20, 2024
@ElijahAhianyo ElijahAhianyo marked this pull request as ready for review March 20, 2024 14:30
@ElijahAhianyo ElijahAhianyo force-pushed the elijah/benchmark-reflex-size branch from 4006423 to 69716ee Compare March 21, 2024 12:40
@ElijahAhianyo ElijahAhianyo force-pushed the elijah/benchmark-reflex-size branch from 69716ee to 0a3dc4c Compare March 21, 2024 12:53
@ElijahAhianyo ElijahAhianyo requested a review from martinxu9 March 22, 2024 10:47
@ElijahAhianyo ElijahAhianyo merged commit e0fedeb into main Mar 25, 2024
66 checks passed
@picklelo picklelo mentioned this pull request Apr 26, 2024
24 tasks
@picklelo picklelo deleted the elijah/benchmark-reflex-size branch August 21, 2024 20:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants