Daniel Lok

Results 93 comments of Daniel Lok

merging in order to make future PRs easier to review / remove need for rebasing. the test failure is simply due to commented out tests, and that can just be...

Thanks for the report and the fix! it makes sense to me

cc @marinegor could you make that PR to `mlflow/mlflow`? it looks like it's currently targeting your fork

discussed offline, we'll update the spacing to use gap instead of min-width so the spacing is consistent, then update the same in `mlflow-website`

i'm going to combine this with #16214 to avoid merge conflicts

thanks for the report @adi250491, since you have the fix ready, would you like to make the PR for it? if you don't have time one of the maintainers can...

note that we do have a util for this, it looks like it just wasn't called for this flavor. if you'd like to make the PR, please follow the pattern...

Thanks for the report, am I understanding correctly that this only happens with newer versions of ONNX?

Got it, the team might not have time to look into this immediately but I'll tag this as `help wanted` in case someone else from the community can contribute a...

Thanks for the report, and the issue make sense! Could you provide a little more information—for example, what seems to be taking up the most space? This will help us...