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

Verify vectorized conditional expressions (if/then/else) #4602

Closed
onelson opened this issue Mar 25, 2022 · 4 comments · Fixed by #5291
Closed

Verify vectorized conditional expressions (if/then/else) #4602

onelson opened this issue Mar 25, 2022 · 4 comments · Fixed by #5291
Assignees

Comments

@onelson
Copy link
Contributor

onelson commented Mar 25, 2022

Following on from #4601.

Once the feature-flagged vectorized implementation has been released, check it out in lovelace to make sure they are working as intended.

Once satisfied, enable the feature flag for everyone, then remove the feature flag.

@onelson
Copy link
Contributor Author

onelson commented Aug 12, 2022

Flipped the flag on in lovelace to collect some metrics.

@onelson
Copy link
Contributor Author

onelson commented Aug 18, 2022

Bumping this one a few sprints down the line for flag removal.

@onelson
Copy link
Contributor Author

onelson commented Sep 22, 2022

Having had to turn this off (twice) for an assortment of issues (several unrelated directly to conditionals), this is back on and the logs have not shown any new defects re: vectors.

I think we're safe to leave this on and prepare to remove the flag in a couple weeks.

@onelson
Copy link
Contributor Author

onelson commented Sep 23, 2022

Nothing new showed up related to vectorized conditionals -- punting this to a future sprint for flag removal.

@onelson onelson removed their assignment Sep 23, 2022
@onelson onelson self-assigned this Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant