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

[pure refactor] "namespace" flag parameters for linestrip & point cloud shader flags #2033

Merged
merged 3 commits into from May 4, 2023

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented May 3, 2023

Also, remove NO_COLOR_GRADIENT in favor of FLAG_COLOR_GRADIENT

(stuff that came up on #2008)

What

Checklist

PR Build Summary: https://build.rerun.io/pr/2033

@Wumpf Wumpf added 🔺 re_renderer affects re_renderer itself 🚜 refactor Change the code, not the functionality labels May 3, 2023
@Wumpf Wumpf changed the title "namespace" flag parameters for linestrip & point cloud shader flags [pure refactor] "namespace" flag parameters for linestrip & point cloud shader flags May 3, 2023
Copy link
Member

@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.

I think I caught at least one place where some gradient appeared where it shouldn't have, there might be others... 👀

@Wumpf
Copy link
Member Author

Wumpf commented May 4, 2023

uhoh. Thanks! I'll look through them again, don't want surprises!

@Wumpf Wumpf merged commit a676bdb into main May 4, 2023
15 checks passed
@Wumpf Wumpf deleted the andreas/re_renderer/some-flag-name-cleanup branch May 4, 2023 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔺 re_renderer affects re_renderer itself 🚜 refactor Change the code, not the functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants