Skip to content

Expose is_stateful in ops #234

Closed
Closed

Description

@rnett

Op protos have a is_stateful, which should be exposed at the instance level in the generated operand classes. This is needed in function defs as StatefulPartitionedCall should be used instead of PartitionedCall to call it.

Activity

  1. Craigacp commented on Mar 6, 2021

    @Craigacp
    Collaborator

    I guess that's the thing we should use to determine if an op needs a strong or a weak reference in eager session.

  2. rnett commented on Mar 6, 2021

    @rnett
    ContributorAuthor

    Maybe? I'm not entirely clear on what it is used for, but that sounds right.

    I've worked around this using the op def list in TensorFlow, and added a more efficient function for it in #233.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions