diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 1ab8d75f0..f09022996 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -1,27 +1,32 @@
--   repo: https://github.com/PaddlePaddle/mirrors-yapf.git
-    sha: 0d79c0c469bab64f7229c9aca2b1186ef47f0e37
-    hooks:
-    -   id: yapf
-        files: \.py$
--   repo: https://github.com/pre-commit/pre-commit-hooks
-    sha: a11d9314b22d8f8c7556443875b731ef05965464
-    hooks:
-    -   id: check-merge-conflict
-    -   id: check-symlinks
-    -   id: detect-private-key
-        files: (?!.*paddle)^.*$
-    -   id: end-of-file-fixer
-        files: \.(md|yml)$
-    -   id: trailing-whitespace
-        files: \.(md|yml)$
--   repo: https://github.com/Lucas-C/pre-commit-hooks
-    sha: v1.0.1
-    hooks:
-    -   id: forbid-crlf
-        files: \.(md|yml)$
-    -   id: remove-crlf
-        files: \.(md|yml)$
-    -   id: forbid-tabs
-        files: \.(md|yml)$
-    -   id: remove-tabs
-        files: \.(md|yml)$
+- repo: https://github.com/PaddlePaddle/mirrors-yapf.git
+  sha: 0d79c0c469bab64f7229c9aca2b1186ef47f0e37
+  hooks:
+    - id: yapf
+      files: \.py$
+
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  sha: a11d9314b22d8f8c7556443875b731ef05965464
+  hooks:
+    - id: flake8
+      args: ['--ignore=E265']
+    - id: check-yaml
+    - id: check-merge-conflict
+    - id: check-symlinks
+    - id: detect-private-key
+      files: (?!.*paddle)^.*$
+    - id: end-of-file-fixer
+      files: \.(md|yml)$
+    - id: trailing-whitespace
+      files: \.(md|yml)$
+
+- repo: https://github.com/Lucas-C/pre-commit-hooks
+  sha: v1.0.1
+  hooks:
+    - id: forbid-crlf
+      files: \.(md|yml)$
+    - id: remove-crlf
+      files: \.(md|yml)$
+    - id: forbid-tabs
+      files: \.(md|yml)$
+    - id: remove-tabs
+      files: \.(md|yml)$
diff --git a/tools/eval.py b/tools/eval.py
index 1c751d95d..ffae03008 100644
--- a/tools/eval.py
+++ b/tools/eval.py
@@ -1,33 +1,31 @@
-#copyright (c) 2020 PaddlePaddle Authors. All Rights Reserve.
+# copyright (c) 2020 PaddlePaddle Authors. All Rights Reserve.
 #
-#Licensed under the Apache License, Version 2.0 (the "License");
-#you may not use this file except in compliance with the License.
-#You may obtain a copy of the License at
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
 #
 #    http://www.apache.org/licenses/LICENSE-2.0
 #
-#Unless required by applicable law or agreed to in writing, software
-#distributed under the License is distributed on an "AS IS" BASIS,
-#WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-#See the License for the specific language governing permissions and
-#limitations under the License.
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 from __future__ import absolute_import
 from __future__ import division
 from __future__ import print_function
 
-import sys
+import os
 import argparse
 
-import paddle
 import paddle.fluid as fluid
 
 import program
 
 from ppcls.data import Reader
 from ppcls.utils.config import get_config
-from ppcls.utils.save_load import init_model, save_model
-from ppcls.utils import logger
+from ppcls.utils.save_load import init_model
 
 from paddle.fluid.incubate.fleet.collective import fleet
 from paddle.fluid.incubate.fleet.base import role_maker
@@ -39,7 +37,7 @@ def parse_args():
         '-c',
         '--config',
         type=str,
-        default='configs/eval.yaml',
+        default='./configs/eval.yaml',
         help='config file path')
     parser.add_argument(
         '-o',
@@ -69,7 +67,7 @@ def main(args):
     exe = fluid.Executor(place)
     exe.run(startup_prog)
 
-    init_model(config, valid_prog, exe, 'ppcls')
+    init_model(config, valid_prog, exe)
 
     valid_reader = Reader(config, 'valid')()
     valid_dataloader.set_sample_list_generator(valid_reader, place)
diff --git a/tools/train.py b/tools/train.py
index 3fad7568d..83790a87e 100644
--- a/tools/train.py
+++ b/tools/train.py
@@ -1,17 +1,16 @@
-#copyright (c) 2020 PaddlePaddle Authors. All Rights Reserve.
+# copyright (c) 2020 PaddlePaddle Authors. All Rights Reserve.
 #
-#Licensed under the Apache License, Version 2.0 (the "License");
-#you may not use this file except in compliance with the License.
-#You may obtain a copy of the License at
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
 #
 #    http://www.apache.org/licenses/LICENSE-2.0
 #
-#Unless required by applicable law or agreed to in writing, software
-#distributed under the License is distributed on an "AS IS" BASIS,
-#WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-#See the License for the specific language governing permissions and
-#limitations under the License.
-
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 from __future__ import absolute_import
 from __future__ import division
@@ -20,7 +19,6 @@ from __future__ import print_function
 import argparse
 import os
 
-import paddle
 import paddle.fluid as fluid
 
 import program
@@ -28,7 +26,6 @@ import program
 from ppcls.data import Reader
 from ppcls.utils.config import get_config
 from ppcls.utils.save_load import init_model, save_model
-from ppcls.utils import logger
 
 from paddle.fluid.incubate.fleet.collective import fleet
 from paddle.fluid.incubate.fleet.base import role_maker
@@ -76,7 +73,7 @@ def main(args):
         # clone to prune some content which is irrelevant in valid_prog
         valid_prog = valid_prog.clone(for_test=True)
 
-    # create the "Executor" with the statement of which place 
+    # create the "Executor" with the statement of which place
     exe = fluid.Executor(place=place)
     # only run startup_prog once to init
     exe.run(startup_prog)
@@ -97,12 +94,12 @@ def main(args):
         # 1. train with train dataset
         program.run(train_dataloader, exe, compiled_train_prog, train_fetchs,
                     epoch_id, 'train')
-        # 2. validate with validate dataset 
+        # 2. validate with validate dataset
         if config.validate and epoch_id % config.valid_interval == 0:
             program.run(valid_dataloader, exe, compiled_valid_prog,
                         valid_fetchs, epoch_id, 'valid')
 
-        # 3. save the persistable model 
+        # 3. save the persistable model
         if epoch_id % config.save_interval == 0:
             model_path = os.path.join(config.model_save_dir,
                                       config.ARCHITECTURE["name"])