I've pushed the mentioned changes to <https://gith...
# hacktoberfest-2022
c
I've pushed the mentioned changes to https://github.com/flyteorg/flytekit/pull/1269/files @Samhita Alla
s
Thanks, @cryptic! Will review the changes and get back to you.
c
I don't have any checks (0 checks) on my PR, is something wrong here?
it says no jobs to run @Eduardo Apolinario (eapolinario)
s
Um yeah; I see that too! Can you do a force push?
c
Okay, so i just updated my fork and it fixed it
s
Hey. We shouldn’t be using
tensor_dtype
as class variable in this case because it needs to be set a different value within
to_literal
method and that won’t be known to
to_python_value
method. If you want to work on this, you’ll need to send
tensor_dtype
as
format
of the blob.
c
I've made the suggested changes, please have a look
s
Thanks!
c
I hope the changes are correct?
s
LGTM! I’ll merge it soon.
157 Views