-
Notifications
You must be signed in to change notification settings - Fork 55
Cast Sequence to list on assignment (with templates) #249
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
base: rolling
Are you sure you want to change the base?
Conversation
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
|
@christophebedard here is an implementation with straight templates. The only jankness comes not being able to apply indents to templates nicely. If there is some cleaner way I would love to know. Trying to use the |
|
Pulls: #249 |
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
|
Pulls: #249 |
Description
With templates
My proposal on what the solution for The Sequence Api discussion.
Zulip discussion here.
Is this user-facing behavior change?
Yes in the past users type would only change on receiving from a subscriber but, now the change is done on assignment to be more inline with numpy.array and array.array casts.
Did you use Generative AI?
Additional Information