keras-team / keras-preprocessing

Utilities for working with image data, text data, and sequence data.
Other
1.02k stars 444 forks source link

A step towards development standardization. #326

Closed eli-osherovich closed 3 years ago

eli-osherovich commented 3 years ago

Configured autoformatter (yapf) to follow the Google python style. Added some entries to .gitignore for VSCode.

Summary

Related Issues

PR Overview

An attempt to standardize development (code style) to minimize spurious diffs.

eli-osherovich commented 3 years ago

@Dref360 Do you have any objections here?

I would like to setup autoformatting (by yapf) to:

  1. keep single code style (google)
  2. get rid of spurious diffs due to different formatting in PRs.
Dref360 commented 3 years ago

Sorry for the delay, I had to verify some things.

In the near future, keras-prepro will be merged back to keras as we do not support multiple backends anymore.

So it doesn't make much sense to push this PR right now since we will have to make it again once we move.

I'll close for now, feel free to open a similar PR on https://github.com/keras-team/keras