Conversation
|
@ngc92 Just one Q about this, in our actual LB evals, how do we enforce a hidden seed that isn't visible to the user? |
|
alexzhang13
left a comment
There was a problem hiding this comment.
I'm happy to merge this because the LBs on our server right now aren't really being used. We can just update the example kernels and the reference code once this is merged, just ping me.
|
I guess the question is whether we should merge this to main, or try to gather more breaking changes and merge them all at once. |
|
@ngc92 I think it's fine to merge in, at this point no one is really iterating on the LBs on the submission side so breaking changes are ok. As long as we inform people on the server it should be fine -- I think a lot of ppl are busy right now anyways so not much has been going on. |
adds a seed argument to data generation functions.
NOTE This requires an update to existing leaderboard reference files
Here, I've only updated the identity test so far.