Skip to content

update bloom model to use model loader from tt-forge repoistory #1007

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.

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ctr-lelanchelianTT
Copy link
Contributor

Problem description

Update JAX tests of bloom to use ModelLoader

What's changed

update Bloom model to use ModelLoader from tt-forge-models

Checklist

  • New/Existing tests provide coverage for changes

I have attached the logs for your reference:
test_bloom_1b1_with_xfail.log
test_bloom_1b1_without_xfail.log
test_bloom_1b7_with_xfail.log
test_bloom_1b7_without_xfail.log
test_bloom_3b_with_xfail.log
test_bloom_3b_without_xfail.log
test_bloom_7b_with_xfail.log
test_bloom_7b_withou_xfail.log
test_bloom_560m_with_xfail.log
test_bloom_560m_without_xfail.log

@ctr-lelanchelianTT ctr-lelanchelianTT self-assigned this Aug 14, 2025
@ctr-lelanchelianTT ctr-lelanchelianTT force-pushed the lelanchelian/update_jax_variants_of_bloom_to_use_forge_models_repo branch from 00c5aac to c84b0a9 Compare August 14, 2025 08:01
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.41%. Comparing base (8fbb97c) to head (40eb91f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1007   +/-   ##
=======================================
  Coverage   68.41%   68.41%           
=======================================
  Files          25       25           
  Lines        1757     1757           
=======================================
  Hits         1202     1202           
  Misses        555      555           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants