Skip to content

Restructure python codegen: move codegen to rerun and old log APIs to log_deprecated#3374

Merged
teh-cmc merged 33 commits intomainfrom
jleibs/reorganize_python_codegen
Sep 20, 2023
Merged

Restructure python codegen: move codegen to rerun and old log APIs to log_deprecated#3374
teh-cmc merged 33 commits intomainfrom
jleibs/reorganize_python_codegen

Conversation

@jleibs
Copy link
Copy Markdown
Contributor

@jleibs jleibs commented Sep 19, 2023

What

Moves:

  • rerun.log.error_utils -> rerun.error_utils
  • rerun.log.* -> rerun.log_deprecated.*
  • rerun.components.* -> rerun.components_deprecated.*
  • _rerun2.* -> rerun.*

Note, this required making the following changes in a few examples:

  • rr.log.rects.RectFormat -> rr.RectFormat
  • rr.log.text.LoggingHandler -> rr.LoggingHandler
  • rr.log.annotation.AnnotationInfo -> rr.AnnotationInfo

Additionally, all of the AffixFuzzer is now generated into its own module in rerun_py.tests.test_types

Does not yet move the new logging APIs out of experimental, but all of that can be managed in rerun.__init__ now.

Checklist

@jleibs jleibs added codegen/idl sdk-python Python logging API labels Sep 19, 2023
@jleibs jleibs marked this pull request as ready for review September 19, 2023 23:52
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not technically a new file. Git is just confused because there's a new components/__init__.py in its place.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not technically a new file. Git is just confused because there's a new components/transform3d.py in its place.

@teh-cmc teh-cmc self-requested a review September 20, 2023 07:39
Copy link
Copy Markdown
Contributor

@teh-cmc teh-cmc left a comment

Choose a reason for hiding this comment

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

based

@teh-cmc teh-cmc merged commit b2f7f19 into main Sep 20, 2023
@teh-cmc teh-cmc deleted the jleibs/reorganize_python_codegen branch September 20, 2023 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codegen/idl sdk-python Python logging API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Exclude mock archetypes/datatypes/components from release artifacts

2 participants