Unverified Commit ca37db05 authored by Stas Bekman's avatar Stas Bekman Committed by GitHub
Browse files

[flax] fix repo_check (#7914)

* [flax] fix repo_check

Unless, this is actually a problem, this adds `modeling_flax_utils` to ignore list. otherwise currently it expects to have a 'tests/test_modeling_flax_utils.py' for it.
for context please see: https://github.com/huggingface/transformers/pull/3722#issuecomment-712360415

* fix 2 more issues

* merge https://github.com/huggingface/transformers/pull/7919/
parent 048dd6cf
...@@ -264,7 +264,7 @@ jobs: ...@@ -264,7 +264,7 @@ jobs:
- v0.3-{{ checksum "setup.py" }} - v0.3-{{ checksum "setup.py" }}
- run: pip install --upgrade pip - run: pip install --upgrade pip
- run: pip install isort - run: pip install isort
- run: pip install .[tf,torch,quality] - run: pip install .[tf,torch,flax,quality]
- save_cache: - save_cache:
key: v0.3-code_quality-{{ checksum "setup.py" }} key: v0.3-code_quality-{{ checksum "setup.py" }}
paths: paths:
......
...@@ -41,12 +41,14 @@ IGNORE_NON_TESTED = [ ...@@ -41,12 +41,14 @@ IGNORE_NON_TESTED = [
# trigger the common tests. # trigger the common tests.
TEST_FILES_WITH_NO_COMMON_TESTS = [ TEST_FILES_WITH_NO_COMMON_TESTS = [
"test_modeling_camembert.py", "test_modeling_camembert.py",
"test_modeling_flax_bert.py",
"test_modeling_flax_roberta.py",
"test_modeling_mbart.py",
"test_modeling_pegasus.py",
"test_modeling_tf_camembert.py", "test_modeling_tf_camembert.py",
"test_modeling_tf_xlm_roberta.py", "test_modeling_tf_xlm_roberta.py",
"test_modeling_xlm_prophetnet.py", "test_modeling_xlm_prophetnet.py",
"test_modeling_xlm_roberta.py", "test_modeling_xlm_roberta.py",
"test_modeling_pegasus.py",
"test_modeling_mbart.py",
] ]
# Update this list for models that are not documented with a comment explaining the reason it should not be. # Update this list for models that are not documented with a comment explaining the reason it should not be.
...@@ -88,6 +90,7 @@ def get_model_modules(): ...@@ -88,6 +90,7 @@ def get_model_modules():
"modeling_outputs", "modeling_outputs",
"modeling_retribert", "modeling_retribert",
"modeling_utils", "modeling_utils",
"modeling_flax_utils",
"modeling_transfo_xl_utilities", "modeling_transfo_xl_utilities",
"modeling_tf_auto", "modeling_tf_auto",
"modeling_tf_outputs", "modeling_tf_outputs",
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment