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.

Sign up for GitHub

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

Jump to bottom

Track compile-times and dependency count of `rerun-sdk` over time #5101

Closed

emilk opened this issue Feb 7, 2024 · 0 comments · Fixed by #5228

Labels

🧑‍💻 dev experience

Member

emilk commented Feb 7, 2024

Part of Minimize crate deps of the logging SDK to improve compile times #4788

emilk added the 🧑‍💻 dev experience label

emilk mentioned this issue

Remove egui_plot as dependency from re_sdk #5099

Merged

5 tasks

teh-cmc added a commit that referenced this issue


          Remove egui_plot as dependency from re_sdk (#5099)

a0467a4

### What
Part of #4788

We need to start tracking the number of dependencies and/or compile-time
of `re_sdk`: #5101.

@v-morlock pointed out in
https://github.com/rerun-io/rerun/issues/5095we accidentally pull in all
of `egui` 🤦

This PR fixes that.

---------

Co-authored-by: Clement Rey <[email protected]>

emilk mentioned this issue

Track crate dependency count over time #5228

Merged

5 tasks

emilk closed this as completed in #5228

emilk added a commit that referenced this issue


          Track crate dependency count over time (#5228)

### What
* Part of #4788
* Closes #5101

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested the web demo (if applicable):
* Using newly built examples:
[app.rerun.io](https://app.rerun.io/pr/5228/index.html)
* Using examples from latest `main` build:
[app.rerun.io](https://app.rerun.io/pr/5228/index.html?manifest_url=https://app.rerun.io/version/main/examples_manifest.json)
* Using full set of examples from `nightly` build:
[app.rerun.io](https://app.rerun.io/pr/5228/index.html?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG
* [x] If applicable, add a new check to the [release
checklist](https://github.com/rerun-io/rerun/blob/main/tests/python/release_checklist)!

- [PR Build Summary](https://build.rerun.io/pr/5228)
- [Docs
preview](https://rerun.io/preview/0223b21104bad205cfc5004aa33386d9b52fd603/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/0223b21104bad205cfc5004aa33386d9b52fd603/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)

---------

Co-authored-by: jprochazk <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment