Skip to content

Export ops.pbtxt in addition to ops.pb - #306

Merged
karllessard merged 3 commits into
tensorflow:masterfrom
rnett:rn_generate_ops_pbtxt
Apr 30, 2021
Merged

Export ops.pbtxt in addition to ops.pb#306
karllessard merged 3 commits into
tensorflow:masterfrom
rnett:rn_generate_ops_pbtxt

Conversation

@rnett

Copy link
Copy Markdown
Contributor

#305, but done properly. The ops.pbtxt isn't used by the generator, but it's good to have as documentation and for git diffs.

…sting
Signed-off-by: Ryan Nett <rnett@calpoly.edu>
@rnett

Copy link
Copy Markdown
ContributorAuthor

@karllessard this needs CI Build.

@rnett

Copy link
Copy Markdown
ContributorAuthor

The failure looks like an actions issue, it's working on some platforms and I didn't change anything platform-specific.

@karllessard

Copy link
Copy Markdown
Collaborator

Hey @rnett , there is a lot of undesirable reformatting in your PR, can you please turn it off and resubmit?

rnett added 2 commits April 29, 2021 14:01
Signed-off-by: Ryan Nett <JNett96@gmail.com>
Signed-off-by: Ryan Nett <JNett96@gmail.com>
@karllessard

Copy link
Copy Markdown
Collaborator

yeah ok, something seems wrong on the Windows platform but yeah, probably unrelated... (sigh)

let's merge then

@karllessard
karllessard merged commit af09c0f into tensorflow:masterApr 30, 2021
Sign up for freeto 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.

3 participants

@rnett@karllessard@Craigacp