Skip to content

Conversation

yongtang
Copy link
Owner

@yongtang yongtang commented Feb 6, 2022

Signed-off-by: Yong Tang [email protected]

version = list(e[10:] for e in entries if e.startswith("version = "))[0].strip('"')
require = list(e[10:] for e in entries if e.startswith("require = "))[0].strip('"')
assert version != ""
assert version != ""
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change
assert version != ""
assert version != ""

Signed-off-by: Yong Tang <[email protected]>

# Note: zlib is placed earlier as tensorflow's zlib does not include unzip
http_archive(

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change

http_archive(

name = "zlib",

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change

Comment on lines +17 to +18


Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change

namespace tensorflow {
namespace io {


Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change

class DecodeAVIFOp : public OpKernel {
public:
explicit DecodeAVIFOp(OpKernelConstruction* context) : OpKernel(context) {}
explicit DecodeAVIFOp(OpKernelConstruction* context) : OpKernel(context) {}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[reviewdog-suggester] reported by reviewdog 🐶

Suggested change
explicit DecodeAVIFOp(OpKernelConstruction* context) : OpKernel(context) {}
explicit DecodeAVIFOp(OpKernelConstruction* context) : OpKernel(context) {}

@yongtang yongtang closed this Feb 6, 2022
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.

1 participant