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.8.1 #3021

Closed
wants to merge 16 commits into from
Closed

Release 0.8.1 #3021

wants to merge 16 commits into from

Conversation

emilk
Copy link
Member

@emilk emilk commented Aug 17, 2023

Changelog

🐍 Python SDK

  • Add a warning category and stacklevel to rerun warnings.warn calls #2985

🪳 Bug Fixes

  • Fix always redrawing in the presence of a 3D space view #2900
  • Fix unable to set camera spinning until camera has moved #2990

🌁 Viewer Improvements

  • Allow changing plot aspect ratio with scroll + cmd/ctrl + alt #2742
  • Automatically select user timeline if no timeline was explicitly selected yet #2986

🧑‍🏫 Examples

  • Add Helix to demo.rerun.io #2930

📈 Analytics

  • Make sure re_analytics never log higher than at debug level #3014

Checklist

Wumpf and others added 9 commits August 17, 2023 13:14
* Fixes #2897

### What

Fix regression introduced in #2839
This landed in 0.8. Candidate for potential 0.8.1 release if we decide
to do one.

### 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/2900) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2900)
- [Docs
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-redraw/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-redraw/examples)
<!--
Open the PR up as a draft until you feel it is ready for a proper
review.

Do not make PR:s from your own `main` branch, as that makes it difficult
for reviewers to add their own fixes.

Add any improvements to the branch as new commits to make it easier for
reviewers to follow the progress. All commits will be squashed to a
single commit once the PR is merged into `main`.

Make sure you mention any issues that this PR closes in the description,
as well as any other related issues.

To get an auto-generated PR description you can put "copilot:summary" or
"copilot:walkthrough" anywhere.
-->

### What

Wanted Shift+Cmd originally, but egui no longer see this as a scroll
then. Also, shift+scroll is already taken for translating along y.

Fixes #2148
* #2148



https://github.com/rerun-io/rerun/assets/1220815/bc078094-4bae-4980-8508-af0cff234614



### 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/2742) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2742)
- [Docs
preview](https://rerun.io/preview/pr%3Aandreas%2Fsingle-axis-plot-zoom/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aandreas%2Fsingle-axis-plot-zoom/examples)
What the title says.

Didn't rename the `dna` folder on purpose: not particularly looking to
deal with broken links all over the place today :)

### What

### 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/2930) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2930)
- [Docs preview](https://rerun.io/preview/pr%3Acmc%2Fhelix_demo/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Acmc%2Fhelix_demo/examples)
…2985)

Resolves: #2408

### What:
Stop overriding the global `warnings.formatwarning`

Instead introduce a RerunWarning category and also set stacklevel
appropriately for better warning context.

Consider this example:
```
import warnings
import rerun as rr

warnings.warn("my warning")
rr.log_points("foo", [1, 2, 3])
rr.init("foo")
rr.log_points("foo", "not a point")
```

Before: wrongly annotated user-warning. No way to identify source of
error.
```
WARNING:rerun:my warning
WARNING:rerun:Rerun is disabled - log_points() call ignored. You must call rerun.init before using log APIs.
WARNING:root:Ignoring rerun log call: Traceback (most recent call last):
  File "/home/jleibs/venv/lib/python3.10/site-packages/rerun_sdk/rerun/log/log_decorator.py", line 47, in wrapper
    return func(*args, **kwargs)
  File "/home/jleibs/venv/lib/python3.10/site-packages/rerun_sdk/rerun/log/points.py", line 228, in log_points
    positions = np.require(positions, dtype="float32")
  File "/home/jleibs/venv/lib/python3.10/site-packages/numpy/core/_asarray.py", line 110, in require
    return asanyarray(a, dtype=dtype)
ValueError: could not convert string to float: 'not a point'
```

After: Consistent pythonic warnings and actionable line-numbers.
```
/home/jleibs/rerun/test.py:4: UserWarning: my warning
  warnings.warn("my warning")
/home/jleibs/rerun/test.py:5: RerunWarning: Rerun is disabled - log_points() call ignored. You must call rerun.init before using log APIs.
  rr.log_points("foo", [1, 2, 3])
/home/jleibs/rerun/test.py:7: RerunWarning: Ignoring rerun log call: Traceback (most recent call last):
  File "/home/jleibs/rerun/rerun_py/rerun_sdk/rerun/log/log_decorator.py", line 51, in wrapper
    return func(*args, **kwargs)
  File "/home/jleibs/rerun/rerun_py/rerun_sdk/rerun/log/points.py", line 208, in log_points
    positions = np.require(positions, dtype="float32")
  File "/home/jleibs/rerun/venv/lib/python3.10/site-packages/numpy/core/_asarray.py", line 106, in require
    return asanyarray(a, dtype=dtype)
ValueError: could not convert string to float: 'not a point'

  rr.log_points("foo", "not a point")
```

### 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/2985) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2985)
- [Docs
preview](https://rerun.io/preview/pr%3Ajleibs%2Frerun_warnings/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Ajleibs%2Frerun_warnings/examples)
Fixes #2892

Thought this was related to blueprint system overwriting, but it was
something else entirely actually: The "interpolate to"-method would set
spin to false and we would interpolate to whatever the latest default
position of the camera is until it is moved first.

* [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/2990) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2990)
- [Docs
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-non-sticky-spinbox/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-non-sticky-spinbox/examples)
…cted yet (#2986)

* Fixes #2186

Slight deviation from the desired behavior from the ticket: We'll always
select the user timeline which is _alphabetically first_. I deem this
good enough though and an improvement in any case!

Going back the the "auto select" state is not possible explicitely, but
will happen if for whatever reason no user timeline is available
anymore.

Tested as seen in the video and then confirmed separately the described
alphabetic behavior



https://github.com/rerun-io/rerun/assets/1220815/0d3303d7-7b19-4854-993d-0523b3e35cbb



### 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/2986) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2986)
- [Docs
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-timeline-auto-select/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aandreas%2Ffix-timeline-auto-select/examples)
* closes #2965

### 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/3015) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/3015)
- [Docs
preview](https://rerun.io/preview/pr%3Aemilk%2Fpublicize-conda/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aemilk%2Fpublicize-conda/examples)
@emilk emilk changed the base branch from main to release-0.8 August 17, 2023 11:30
@emilk emilk added ⛴ release Related to shipping or publishing exclude from changelog PRs with this won't show up in CHANGELOG.md labels Aug 17, 2023
@emilk emilk changed the base branch from release-0.8 to main August 17, 2023 12:22
@emilk emilk changed the base branch from main to release-0.8 August 17, 2023 12:25
@emilk emilk marked this pull request as ready for review August 17, 2023 14:40
@emilk emilk added this to the 0.8.1 milestone Aug 17, 2023
emilk added a commit that referenced this pull request Aug 17, 2023
… and other fixes that surfaced during the release of 0.8.1 in
#3021.

### 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/3029) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/3029)
- [Docs
preview](https://rerun.io/preview/pr%3Aemilk%2F0.8.1-changelog/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Aemilk%2F0.8.1-changelog/examples)
@emilk
Copy link
Member Author

emilk commented Aug 17, 2023

All commits have been cherry-picked into https://github.com/rerun-io/rerun/commits/release-0.8

@emilk emilk closed this Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants