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

Release 0.11 #4347

Merged
merged 35 commits into from
Nov 28, 2023
Merged

Release 0.11 #4347

merged 35 commits into from
Nov 28, 2023

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Nov 27, 2023

What

TODO for final release:

TODO just after release:

  • Update webpage with blog
  • Merge opencv/eigen example update
  • github release update
  • post on social

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested app.rerun.io (if applicable)
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

@Wumpf Wumpf added the ⛴ release Related to shipping or publishing label Nov 27, 2023
Wumpf and others added 11 commits November 27, 2023 16:53
* Closes #4315

Adds:
* Open Photogrammetry Example
* External C++ VRS example
* External C++ Eigen/OpenCV exaple

I also opted to sort the examples in lexicographical order because it's
in my DNA to do so.

Check the "Examples preview" below

### 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 [demo.rerun.io](https://demo.rerun.io/pr/4342) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4342)
- [Docs
preview](https://rerun.io/preview/1598cc75e9a913ae43866e2ab570ea9cce0d4110/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/1598cc75e9a913ae43866e2ab570ea9cce0d4110/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)
@nikolausWest
Copy link
Member

Testing the python wheels:

The manifest seems to be wrong / rrd files not uploaded to the right place. When I click an example it doesn't load and gives this error:
Screenshot 2023-11-27 at 21 23 34

@Wumpf
Copy link
Member Author

Wumpf commented Nov 27, 2023

The manifest seems to be wrong / rrd files not uploaded to the right place.

That's this is related to the rrd upload issue we had during the release ci, should be resolved on the next RC. But I'm very confused now to why this works on the Rust build I tried 🤔

Wumpf and others added 6 commits November 28, 2023 09:25
…ity (#4354)

### What

Was accidentally looking up groups as individual entities. In particular
this broke visible history configuration on a group selection.

### 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 [app.rerun.io](https://app.rerun.io/pr/4354) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4354)
- [Docs
preview](https://rerun.io/preview/200482e934335f0c7169e62aa934e1011b996366/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/200482e934335f0c7169e62aa934e1011b996366/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)
### What
Empty is a more robust check than null since it also catches the empty
string.

I'm torn on whether to also check for null here -- my inclination is not
to since I'd rather raise the unexpected null error since if someone is
passing an StringView with a nullptr but non-zero length, they are doing
something wrong.

* closes #4348

### 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 [app.rerun.io](https://app.rerun.io/pr/4349) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

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

Running cmake for an unziped rerun c++ sdk package would yield this
warning:

```
CMake Warning (dev) in CMakeLists.txt:
  No project() command is present.  The top-level CMakeLists.txt file must
  contain a literal, direct call to the project() command.  Add a line of
  code such as

    project(ProjectName)

  near the top of the file, but after cmake_minimum_required().

  CMake is pretending there is a "project(Project)" command on the first
  line.
This warning is for project developers.  Use -Wno-dev to suppress it.
```

looks like there's no harm in just calling `project` again, even if the
cmake file is in the context of the our repo

### 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 [app.rerun.io](https://app.rerun.io/pr/4350) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

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

Needed for cmake install on windows if there's spaces on any of the
relevant paths.
This happens because unlike when using a cmake variable, here the
@value@ identifiers are replaced with the actual variable value which
may contain spaces

### 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 [app.rerun.io](https://app.rerun.io/pr/4351) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4351)
- [Docs
preview](https://rerun.io/preview/21599dd24a1940681fee718be930ecedeb9577f4/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/21599dd24a1940681fee718be930ecedeb9577f4/examples)
<!--EXAMPLES-PREVIEW-->
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)
@rerun-bot
Copy link
Collaborator

Version 0.11.0-rc.2 published successfully.

artifact install
web app
wheels pip install rerun-sdk==0.11.0-rc.2
crates cargo install rerun-cli@0.11.0-rc.2
npm npm install @rerun-io/web-viewer@0.11.0-rc.2
docs
py docs
rs docs
cpp_sdk zip

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@jprochazk
Copy link
Member

working on a code sandbox example to add to the JS docs

@jprochazk jprochazk added the exclude from changelog PRs with this won't show up in CHANGELOG.md label Nov 28, 2023
@jprochazk
Copy link
Member

JS build will always fail because it starts sooner than the npm publish finishes. It succeeds if we restart it once the publish is done

@rerun-bot
Copy link
Collaborator

Version 0.11.0 published successfully.

artifact install
web app
wheels pip install rerun-sdk==0.11.0
crates cargo install rerun-cli@0.11.0
npm npm install @rerun-io/web-viewer@0.11.0
docs
py docs
rs docs
cpp_sdk zip

@jprochazk
Copy link
Member

What were the biggest issues in this release? From my perspective it was:

  • setup-pixi failing
  • crates.io index race condition

There were some smaller snags in the new release stuff (e.g. npm publish failing), but the two above caused the most problems...

@Wumpf
Copy link
Member Author

Wumpf commented Nov 28, 2023

agreed! The issues we had on our side were all annoying but don't concern me that much. We primarily need to improve a bit on on our internal documentation so it's easier to troubleshoot if @jprochazk is not around 😄. And release more often to get more kinks out. But overall our tooling has been great I think

@Wumpf Wumpf closed this Nov 28, 2023
@Wumpf Wumpf reopened this Nov 28, 2023
@Wumpf Wumpf added the 🧑‍💻 dev experience developer experience (excluding CI) label Nov 28, 2023
@Wumpf Wumpf merged commit dda5c48 into main Nov 28, 2023
42 of 44 checks passed
@Wumpf Wumpf deleted the release-0.11.0 branch November 28, 2023 14:24
@jprochazk jprochazk restored the release-0.11.0 branch November 28, 2023 15:00
teh-cmc pushed a commit that referenced this pull request Nov 30, 2023
### What

* Fixes #4267

TODO for final release:
* [x] Update links, fix all `?speculative-link`
* [x] cherry pick #4349
* [x] cherry pick #4350
* [x] cherry pick #4351
* [x] cherry pick #4354
* [x] Write changelog (and review thereof)
  * Known limitation of Visible History worth mentioning:
     * [x] #4270
     * [x] #723
* [x] testing:
   * special care this time about:
* Blueprint error messages (use outdated stuff you have on your machine
to stretch it!)
* UI changes (selection panel, visual history things - in particular on
the plot)
* C++ cmake install (follow public instructions - if they don't help the
instructions are bad!)
   * [x] Windows
      * [x] Native
           * blueprint ok 0.10.0-rc.1 -> 0.11-rc.1
* found spurious wgpu related crash, seems to be fixed on latest wgpu.
Details on Slack
           * C++ visual studio project for unziped sdk works
* Cmake install works. With path with spaces after applying
#4351
      * [x] Browser
   * [x] Linux
      * [x] Native
      * [x] Browser
   * [x] Mac
       * [x] Native
         * [x] blueprint ok 0.10.1 -> 0.11
         * [x] UI changes ok
       * [x] Browser
   * [x] python package (any platform)
       * [x] tested on mac, seems ok except:
       * [x] ~~couldn't load example rrds~~ fixed 
   * [x] C++ cmake install (any platform)
        * tested on linux 
   * [x] Rust crate (any platform)
* [x] finish opencv/eigen example update

TODO just after release:
* [x] Update webpage with blog
* [x] Merge opencv/eigen example update
* [x] github release update
* [x] post on social

### 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 [app.rerun.io](https://app.rerun.io/pr/4347) (if
applicable)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG

- [PR Build Summary](https://build.rerun.io/pr/4347)
- [Docs
preview](https://rerun.io/preview/01b4186b37fc910491879dfd90433aeb940a5c89/docs)
<!--DOCS-PREVIEW-->
- [Examples
preview](https://rerun.io/preview/01b4186b37fc910491879dfd90433aeb940a5c89/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: Emil Ernerfeldt <emil.ernerfeldt@gmail.com>
Co-authored-by: rerun-bot <bot@rerun.io>
Co-authored-by: jprochazk <1665677+jprochazk@users.noreply.github.com>
Co-authored-by: Jeremy Leibs <jeremy@rerun.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 dev experience developer experience (excluding CI) exclude from changelog PRs with this won't show up in CHANGELOG.md ⛴ release Related to shipping or publishing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release Rerun 0.11
8 participants