-
Notifications
You must be signed in to change notification settings - Fork 732
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
Session 5; provided pre-trained model incompatible (and not loaded) #78
Comments
I ran into a similar error but TF complained about a different, more interpretable issue:
When I changed n_cells from 512 to 200, the error went away, but I started getting mostly garbage output again. |
Ah, I forgot to make the "os.path.exists(ckpt_name + '.index')" fix at inference time. If I add that after initialization I'm seeing reasonable output (well, as reasonable as one can expect in this case). |
Is this working now then or is there still some issue? |
It works for me with the following changes (but doesn't work as-is):
|
New model provided in 5ef9564 |
Sorry this is still half an issue: the new model is compatible, but in |
On Session 5, part 2, the check
... does not match the provided checkpoint file name exactly
trump.ckpt.data-00000-of-00001
so the load is not attempted.Therefore, the model is untrained for the example and the predictions are all non-deterministic random strings,
etc.
If the check is removed, then the
saver
does pick up the provided model, but then cannot load it.It blows up with the error below. I've also noticed, while chasing this, that the
encoder
anddecoder
used for this part of the exercise are actually initialized with a different text -- the one from Part 4 - Character-Level Language Model; I guess that's fine as the sets are probably equivalent in this case, but wouldn't it be better to add a cell just to regenerate these for the latter section?I'm running TF 1.1.0 wit GPU support on a Debian 4.9.18-1 laptop.
The text was updated successfully, but these errors were encountered: