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

use var name to get variable instead of index #2980

Closed
jacquesqiao opened this issue Jul 20, 2017 · 1 comment
Closed

use var name to get variable instead of index #2980

jacquesqiao opened this issue Jul 20, 2017 · 1 comment
Assignees

Comments

@jacquesqiao
Copy link
Member

Now in Operator or Kernel's Run/Infershape function, we may use index such as Input(0)/Ouptput(1) to get variable from scope, there are several problem of use index:

  1. We have to remember what the meaning of each index is. In OpProto, we declare the variable name and the name have it's meaning, such as "weight" "bias". Index do not have meaning and is hard to remember.
  2. For variant input/output, we do not know the real index of the variable we want. It's hard to get what we want with index.

so we should use name declared in OpProto to get var from scope.

@jacquesqiao jacquesqiao self-assigned this Jul 20, 2017
@typhoonzero
Copy link
Contributor

Closing. I think it's already implemented.

heavengate pushed a commit to heavengate/Paddle that referenced this issue Aug 16, 2021
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

No branches or pull requests

2 participants