Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Update helper.py #211

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update helper.py #211

wants to merge 4 commits into from

Conversation

syed-ahmed
Copy link

Hi @dennybritz , I had to make these proposed changes to make the ScheduledOutputSampling work. I was getting Can't convert bool to float 32 error before adding maybe_concatenate_auxiliary_inputs(outputs).cell_output to line 420. I also got InvalidArgumentError: TensorArray dtype is int32 but Op is trying to write dtype bool. before making the changes to line 387, 388 and the casting in 420. Please advice if the changes are valid.

Hi @dennybritz, I had to make these proposed changes to make the ScheduledOutputSampling work. I was getting `Can't convert bool to float 32` error before adding `maybe_concatenate_auxiliary_inputs(outputs).cell_output` to line 420. I also got `InvalidArgumentError: TensorArray dtype is int32 but Op is trying to write dtype bool.` before making the changes to line 387, 388 and the casting in 420. Please advice if the changes are valid.
Adding changes from @graehls pull request 'cause tests were failing.
Adding changes from @graehls pull request 'cause tests were failing.
Adding changes from @graehl's pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant