Closed TheaperDeng closed 3 months ago
LGTM
I guess this will be used within the retrain script?
I guess this will be used within the retrain script?
Yes, I will merge this PR after some unit-test
I guess this will be used within the retrain script?
Yes, I will merge this PR after some unit-test
Just noticed this. Otherwise I am thinking why MT re-training is running so long...
So I guess we don't need to have this data_length
anymore?
So I guess we don't need to have this
data_length
anymore?
Yes, I have removed it from the paramters
Description
1. Motivation and Context
Sometimes we need a subsetsampler to use a subset for the benchmark. This is an easy implementation to make a subset by a sampler.
2. Summary of the change
subsetsampler
for future usage--train_subset <int>
default to 5000 for thedattri_retrain
command.3. What tests have been added/updated for the change?