-
Notifications
You must be signed in to change notification settings - Fork 541
fix: Update metadata to include active number of parameter to ModelMeta
#3837
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
Merged
Samoed
merged 32 commits into
embeddings-benchmark:main
from
ayush1298:add_active_parameter
Jan 19, 2026
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
f2b424b
Add active parameter column on LB
ayush1298 fa8e3cc
update ModelMeta with parameters
ayush1298 e2a296d
update ModelMeta of models
ayush1298 8a884ec
Delete parameter_update_results.csv
ayush1298 6aa26ff
fix test
ayush1298 8081c7e
fix tests
ayush1298 50e5e81
delete script
ayush1298 92bd456
rename for consistency
ayush1298 3db3369
convert active_parameter to property
ayush1298 a2ea58d
Merge branch 'main' into add_active_parameter
ayush1298 222aae5
rename and fix property
ayush1298 bb4fa46
update embedding parameters for model2vec models
ayush1298 a541b55
remove duplicate loading of models
ayush1298 359e1ef
fix
ayush1298 59b5406
lintter
ayush1298 51a60a5
fix
ayush1298 3ce7779
remove separate method for embedding parameter calculation
ayush1298 659845b
fix embedding calculation to pass typecheck
ayush1298 77a0d58
lintter
ayush1298 71ed72c
fix checking
Samoed d757637
Merge branch 'main' into add_active_parameter
Samoed 9846d7c
rename active parameters
Samoed 6f81286
upd docstring
Samoed ec1b1bd
fix tests
ayush1298 7ebdad4
remove n_active_parameters_override from ModelMeta of all models
ayush1298 d69dd0d
lintter
ayush1298 7eb555b
rename file instead of merging main
ayush1298 3a2e37a
Merge branch 'main' into add_active_parameter
ayush1298 e151a30
fix tests
ayush1298 797a805
correct tests
ayush1298 7896319
Delete model total and active parameters - model_parameters.csv
ayush1298 9cbabb5
Merge branch 'main' into add_active_parameter
Samoed File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.