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

Update remote store id field name #8626

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update remote store id field name #8626

wants to merge 1 commit into from

Conversation

zehiko
Copy link
Contributor

@zehiko zehiko commented Jan 9, 2025

What

We've added prefix to all remote store catalog fields and renamed the recording id field, hence updating python SDK and relevant python example.

Testing done

pixi run -e examples -- python examples/python/remote/metadata.py register file:///tmp/structure_from_motion.rrd 

Registered new recording with ID: dd3d7c47-3f04-4d12-9d9b-69b223ee9f56

pixi run -e examples -- python examples/python/remote/metadata.py print
┌────────────────┬────────────────┬────────────────┬───────────────┬───┬───────────────┬───────────────┬───────────────┬───────┐
│ row_id         ┆ rerun_registra ┆ rerun_recordin ┆ rerun_applica ┆ … ┆ rerun_descrip ┆ rerun_storage ┆ rerun_recordi ┆ extra │
│ ---            ┆ tion_time      ┆ g_id           ┆ tion_id       ┆   ┆ tion          ┆ _url          ┆ ng_type       ┆ ---   │
│ struct[2]      ┆ ---            ┆ ---            ┆ ---           ┆   ┆ ---           ┆ ---           ┆ ---           ┆ i64   │
│                ┆ datetime[ns]   ┆ str            ┆ str           ┆   ┆ str           ┆ str           ┆ cat           ┆       │
╞════════════════╪════════════════╪════════════════╪═══════════════╪═══╪═══════════════╪═══════════════╪═══════════════╪═══════╡
│ {1736421859678 ┆ 2025-01-09 11: ┆ dd3d7c47-3f04- ┆ rerun_example ┆ … ┆               ┆ file:///tmp/s ┆ rrd           ┆ 42    │
│ 872875,3498296 ┆ 24:19.678990   ┆ 4d12-9d9b-69b2 ┆ _structure_fr ┆   ┆               ┆ tructure_from ┆               ┆       │
│ 86…            ┆                ┆ 23…            ┆ om_m…         ┆   ┆               ┆ _mot…         ┆               ┆       │
└────────────────┴────────────────┴────────────────┴───────────────┴───┴───────────────┴───────────────┴───────────────┴───────┘

@zehiko zehiko added exclude from changelog PRs with this won't show up in CHANGELOG.md remote-store remote store gRPC API labels Jan 9, 2025
@zehiko zehiko self-assigned this Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
83a39d2 https://rerun.io/viewer/pr/8626 +nightly +main

Note: This comment is updated whenever you push a commit.

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 remote-store remote store gRPC API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant