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

Define correct toString method for all existing layers #301

Closed
zaleslaw opened this issue Dec 15, 2021 · 1 comment · Fixed by #335
Closed

Define correct toString method for all existing layers #301

zaleslaw opened this issue Dec 15, 2021 · 1 comment · Fixed by #335
Assignees
Milestone

Comments

@zaleslaw
Copy link
Collaborator

A part of layers, like the Dense layer, have a broken toString method which could not print the state of the layer before building its part of TF Graph.

We suggest improving library maintainers' experience during debugging and defining toString method for all layers which will be independent of the stage of model initialization and compilation.

@zaleslaw zaleslaw self-assigned this Dec 15, 2021
@zaleslaw zaleslaw added this to the 0.4 milestone Dec 15, 2021
@zaleslaw
Copy link
Collaborator Author

zaleslaw commented Feb 3, 2022

For example I see in debugger Method threw 'kotlin.UninitializedPropertyAccessException' exception. Cannot evaluate org.jetbrains.kotlinx.dl.api.core.layer.convolutional.Conv1D.toString() before weights and variables initialization and the same for all layers with the variables

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