responsible-ai-toolbox icon indicating copy to clipboard operation
responsible-ai-toolbox copied to clipboard

Fix CI notebook text and vision tests

Open imatiach-msft opened this issue 10 months ago β€’ 1 comments

Description

Fix CI notebook text and vision tests Copilot summary: This pull request includes several updates to the CI workflows and a minor update to the development requirements. The most important changes are the removal of Python 3.8 from the test matrix and the addition of specific conditions for uploading artifacts.

Updates to CI workflows:

  • .github/workflows/CI-e2e-notebooks-text-vision.yml: Removed Python 3.8 from the pythonVersion matrix and added specific conditions for the Upload requirements step. Additionally, updated the naming convention for the screenshot artifact to include matrix variables. [1] [2] [3]
  • .github/workflows/CI-notebook-text.yml: Removed Python 3.8 from the pythonVersion matrix.
  • .github/workflows/CI-notebook-vision.yml: Removed Python 3.8 from the pythonVersion matrix.

Update to development requirements:

  • responsibleai_vision/requirements-dev.txt: Removed the version constraint on tensorflow.

Checklist

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

imatiach-msft avatar Jan 29 '25 17:01 imatiach-msft

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.18%. Comparing base (283ee27) to head (7485001). Report is 1 commits behind head on main.

:exclamation: There is a different number of reports uploaded between BASE (283ee27) and HEAD (7485001). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (283ee27) HEAD (7485001)
unittests 5 1
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2597       +/-   ##
===========================================
- Coverage   87.84%   70.18%   -17.66%     
===========================================
  Files         113       18       -95     
  Lines        6657     1278     -5379     
===========================================
- Hits         5848      897     -4951     
+ Misses        809      381      -428     
Flag Coverage Ξ”
unittests 70.18% <ΓΈ> (-17.66%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Jan 29 '25 18:01 codecov-commenter