search

tensorflow / ngraph-bridge

TensorFlow-nGraph bridge
Other
137 stars 64 forks source link

Cavusmustafa/modular backend review v6 #762

Open cavusmustafa opened 3 years ago

cavusmustafa commented 3 years ago

Requesting some cosmetic changes Please remove commented code, use CamelCase for function names

I'm confused about using CamelCase. Some function names does not use it in bridge now. Is there a plan to update them all?

kanvi-nervana commented 3 years ago

Requesting some cosmetic changes Please remove commented code, use CamelCase for function names

I'm confused about using CamelCase. Some function names does not use it in bridge now. Is there a plan to update them all?

Yes, we will update the function names that don't follow it