xiph / awcy

http://arewecompressedyet.com/
MIT License
72 stars 46 forks source link

Format Python Code #301

Closed master-of-zen closed 3 years ago

master-of-zen commented 3 years ago

This PR formats all python code with black, sorts imports and remove unused.

master-of-zen commented 3 years ago

@vibhoothi Done

tdaede commented 3 years ago

The convexhull_framework is actually directly copied from libaom. A todo would be to actually get it some other way, e.g. a submodule. But because it's copied, I think formatting of it should probably be done upstream in libaom first.

The rest of the formatting looks OK.

vibhoothi commented 3 years ago

I agree with Thomas here, it will be problematic when we try to update the local changes. Yes, we should actually make this a submodule to make the updates easier.

master-of-zen commented 3 years ago

@vibhoothi @tdaede removed commit for convex hull changes, now it's only AWCY code

master-of-zen commented 3 years ago

Good to merge?