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

Proper rustc & LLVM versions #1512

Merged
merged 3 commits into from
Mar 6, 2023

Conversation

teh-cmc
Copy link
Member

@teh-cmc teh-cmc commented Mar 6, 2023

Fixes #1509

Also keeps track of LLVM version while we're at it, since we've already seen many lavapipe/llvmpipe related issues already...


$ rerun --version
rerun 0.2.0 [rustc 1.67.0 (fc594f156 2023-01-24), LLVM 15.0.6] x86_64-unknown-linux-gnu cmc/analytics/rust_version e52d606, built 2023-03-06T10:49:07Z

image

@teh-cmc teh-cmc changed the base branch from main to emilk/rust-sdk-refactor March 6, 2023 11:02
@teh-cmc teh-cmc added the 📊 analytics telemetry analytics label Mar 6, 2023
@Wumpf Wumpf self-requested a review March 6, 2023 11:04
Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@Wumpf Wumpf merged commit 454f78f into emilk/rust-sdk-refactor Mar 6, 2023
@Wumpf Wumpf deleted the cmc/analytics/rust_version branch March 6, 2023 11:10
emilk added a commit that referenced this pull request Mar 6, 2023
* Refactor: move the session log sink to its own file

* Turn LogSink into a trait

* Move spawning of native viewer to `rerun` crate

* Move the `show()` method to `rerun` crate

* re_sdk no longer depend directly on re_viewer

* Prevent re_viewer from accidentally depending on re_web_viewer_server

* Improve doc-strings

* Remove the native_viewer feature from re_sdk

* Remove direct dependency of rerun_py on re_sdk_comms

* Small doc-fixes

* Move `clap` hookup from `re_sdk` to `rerun`

* Move the `session.serve()` method to trait `rerun::WebViewerSessionExt`

* cleanup

* Remove cyclic-dependency-protection because it caused weird warning:

The lib target `re_viewer` in package `re_viewer v0.2.0 (/Users/emilk/code/rerun/rerun/crates/re_viewer)` has the same output filename as the lib target `re_viewer` in package `re_viewer v0.2.0 (/Users/emilk/code/rerun/rerun/crates/re_viewer)`.

* Fix web-viewer

* Fix python build

* Fix remove_dir_all RUSTSEC

❯ cargo update -p tempfile
    Updating crates.io index
    Removing remove_dir_all v0.5.3
    Updating tempfile v3.3.0 -> v3.4.0

* Trim re_sdk dependency list

* Fix doc-tests

* Fix doclinks

* Document rust_version

* Better docs for set_sink

* Hide `global_session()` behind the `global` feature flag

* Fix typos

* Proper rustc & LLVM versions (#1512)

* add rustc & llvm version to build crates

* integrate everywhere

* crash handler too

* Rename "global" feature to "global_session"

* Replace extension trait with `serve_web_viewer` method

---------

Co-authored-by: Clement Rey <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📊 analytics telemetry analytics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Analytics rust_version is meaningless
2 participants