Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix release github actions for RAI vision and text packages #2536

Merged
merged 1 commit into from
Feb 12, 2024
Merged

fix release github actions for RAI vision and text packages #2536

merged 1 commit into from
Feb 12, 2024

Conversation

imatiach-msft
Copy link
Contributor

@imatiach-msft imatiach-msft commented Feb 12, 2024 •
edited
Loading

Description

fix release github actions for RAI vision and text packages

Github actions for text and vision seem to be failing in CI due to errors:

Text github action:

/usr/share/miniconda/envs/test/lib/python3.8/site-packages/torch/_custom_op/impl.py:1052: in error_not_found
    raise ValueError(
E   ValueError: Could not find the operator torchvision::nms. Please make sure you have already registered the operator and (if registered from C++) loaded it via torch.ops.load_library.

Vision github action:

  Attempting uninstall: cryptography
    Found existing installation: cryptography 42.0.2
Error: -12T2024-02-12T15:44:40.9776411Z ##[error]No space left on device : '/home/runner/runners/2.312.0/_diag/pages/08edee05-83f0-428a-ba89-169b6c454d4d_2a5570e3-06cd-5cc4-41e4-bbdaf36dbb4d_1.log'

This PR aims to resolve this by incrementing pycocotools==2.0.6 and making builds more similar to the unit tests github action which is passing:
https://github.com/microsoft/responsible-ai-toolbox/blob/main/.github/workflows/CI-responsibleai-text-vision-pytest.yml

Link to passing responsibleai-vision release:
https://github.com/microsoft/responsible-ai-toolbox/actions/runs/7875199845

Link to passing responsibleai-text release:
https://github.com/microsoft/responsible-ai-toolbox/actions/runs/7876079640

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

@imatiach-msft imatiach-msft force-pushed the ilmat/fix-release-text-vision-scripts branch from 10510bb to 9948171 Compare February 12, 2024 16:57
@github-actions GitHub Actions
Copy link

https://responsibleai.blob.core.windows.net/pullrequest/microsoft/responsible-ai-toolbox/ilmat/fix-release-text-vision-scripts/dashboard/index.html

1 similar comment
@github-actions GitHub Actions
Copy link

https://responsibleai.blob.core.windows.net/pullrequest/microsoft/responsible-ai-toolbox/ilmat/fix-release-text-vision-scripts/dashboard/index.html

@imatiach-msft imatiach-msft force-pushed the ilmat/fix-release-text-vision-scripts branch from 9948171 to be5fbf4 Compare February 12, 2024 18:11
@github-actions GitHub Actions
Copy link

https://responsibleai.blob.core.windows.net/pullrequest/microsoft/responsible-ai-toolbox/ilmat/fix-release-text-vision-scripts/dashboard/index.html

@imatiach-msft imatiach-msft merged commit 66da2c0 into main Feb 12, 2024
61 checks passed
@imatiach-msft imatiach-msft deleted the ilmat/fix-release-text-vision-scripts branch February 12, 2024 19:31
@imatiach-msft imatiach-msft mentioned this pull request Apr 18, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaugup gaugup gaugup approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@imatiach-msft @gaugup