Skip to content

Improve Plans flexibility#3199

Merged
karlhigley merged 2 commits intoOpenMined:masterfrom
tudorcebere:master
Mar 16, 2020
Merged

Improve Plans flexibility#3199
karlhigley merged 2 commits intoOpenMined:masterfrom
tudorcebere:master

Conversation

@tudorcebere
Copy link
Contributor

This PR fixes #3184

Solved:

  • added support when input tensor is also output tensor.
  • added a test case when input tensor is also output tensor.

To do:

  • check for the number of parameters
  • create support for nested structures

…nt status:

* added support when input tensor is also output tensor
* added a test case when input tensor is also output tensor
@tudorcebere tudorcebere requested a review from a team March 15, 2020 17:29
@karlhigley
Copy link
Contributor

@tudorcebere Looks like this has a minor formatting issue! You can fix it by running black . in the root directory of the repo. Also highly recommend installing the commit hooks, which will handle this for you in the future.

@tudorcebere tudorcebere reopened this Mar 15, 2020
@karlhigley karlhigley merged commit 7e91927 into OpenMined:master Mar 16, 2020
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 this pull request may close these issues.

Improve Plans flexibility

2 participants