Skip to content

Commit

Permalink
revert main transformers and disable modernbert
Browse files Browse the repository at this point in the history
  • Loading branch information
IlyasMoutawwakil committed Jan 10, 2025
1 parent a3b2608 commit 613ff7b
Show file tree
Hide file tree
Showing 6 changed files with 37 additions and 34 deletions.
1 change: 0 additions & 1 deletion .github/workflows/test_export_onnx.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ jobs:
- name: Install dependencies for pytorch export
run: |
pip install .[tests,exporters,diffusers]
pip install transformers@git+https://github.com/huggingface/transformers.git
- name: Test with unittest
working-directory: tests
run: |
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/test_export_onnx_cli.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ jobs:
- name: Install dependencies for pytorch export
run: |
pip install .[tests,exporters,diffusers]
pip install transformers@git+https://github.com/huggingface/transformers.git
- name: Test with unittest
working-directory: tests
run: |
Expand Down
40 changes: 20 additions & 20 deletions .github/workflows/test_exporters_slow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,23 +18,23 @@ jobs:

runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies for pytorch export
run: |
pip install .[tests,exporters,diffusers]
- name: Test with unittest
working-directory: tests
run: |
RUN_SLOW=1 pytest exporters -s -m "not tensorflow_test and run_slow" --durations=0
- name: Install dependencies for tensorflow export
run: |
pip uninstall diffusers -y
pip install .[tests,exporters-tf]
- name: Test with unittest
working-directory: tests
run: |
RUN_SLOW=1 pytest exporters -s -m "tensorflow_test and run_slow" --durations=0
- uses: actions/checkout@v2
- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies for pytorch export
run: |
pip install .[tests,exporters,diffusers]
- name: Test with unittest
working-directory: tests
run: |
RUN_SLOW=1 pytest exporters -s -m "not tensorflow_test and run_slow" --durations=0
- name: Install dependencies for tensorflow export
run: |
pip uninstall diffusers -y
pip install .[tests,exporters-tf]
- name: Test with unittest
working-directory: tests
run: |
RUN_SLOW=1 pytest exporters -s -m "tensorflow_test and run_slow" --durations=0
7 changes: 6 additions & 1 deletion .github/workflows/test_onnxruntime.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,12 @@ jobs:
- name: Install transformers ${{ matrix.transformers-version }}
if: ${{ matrix.transformers-version != 'latest' }}
run: |
pip install "transformers==${{ matrix.transformers-version }}" "diffusers<0.32.0"
pip install "transformers==${{ matrix.transformers-version }}"
- name: Install diffusers
if: matrix.transformers-version == '4.36.*'
run: |
pip install "diffusers<0.32.0"
- name: Test with pytest (in series)
run: |
Expand Down
18 changes: 9 additions & 9 deletions optimum/exporters/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -902,15 +902,15 @@ class TasksManager:
"image-classification",
onnx="MobileNetV2OnnxConfig",
),
"modernbert": supported_tasks_mapping(
"feature-extraction",
"fill-mask",
"text-classification",
"multiple-choice",
"token-classification",
"question-answering",
onnx="ModernBertOnnxConfig",
),
# "modernbert": supported_tasks_mapping(
# "feature-extraction",
# "fill-mask",
# "text-classification",
# "multiple-choice",
# "token-classification",
# "question-answering",
# onnx="ModernBertOnnxConfig",
# ),
"mpnet": supported_tasks_mapping(
"feature-extraction",
"fill-mask",
Expand Down
4 changes: 2 additions & 2 deletions tests/exporters/exporters_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@
"mobilenet-v2": "hf-internal-testing/tiny-random-MobileNetV2Model",
"mobilenet-v1": "google/mobilenet_v1_0.75_192",
"mobilevit": "hf-internal-testing/tiny-random-mobilevit",
"modernbert": "hf-internal-testing/tiny-random-ModernBertForMaskedLM",
# "modernbert": "hf-internal-testing/tiny-random-ModernBertForMaskedLM",
"mpnet": "hf-internal-testing/tiny-random-MPNetModel",
"mpt": "hf-internal-testing/tiny-random-MptForCausalLM",
"mt5": "lewtun/tiny-random-mt5",
Expand Down Expand Up @@ -267,7 +267,7 @@
# "mobilenet_v1": "google/mobilenet_v1_0.75_192",
# "mobilenet_v2": "google/mobilenet_v2_0.35_96",
"mobilevit": "apple/mobilevit-small",
"modernbert": "answerdotai/ModernBERT-base",
# "modernbert": "answerdotai/ModernBERT-base",
"mpt": "mosaicml/mpt-7b",
"mt5": "lewtun/tiny-random-mt5", # Not using google/mt5-small because it takes too much time for testing.
"musicgen": "facebook/musicgen-small",
Expand Down

0 comments on commit 613ff7b

Please sign in to comment.