Skip to content
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

Add 'format' arg to get_result_df #885

Merged
merged 3 commits into from
Nov 29, 2022

Conversation

loomlike
Copy link
Collaborator

Signed-off-by: Jun Ki Min 42475935+loomlike@users.noreply.github.com

Description

Add format arg back to get_result_df utility for backward compatibility

How was this PR tested?

Unit-test

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
@loomlike loomlike added the safe to test Tag to execute build pipeline for a PR from forked repo label Nov 28, 2022
Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
xiaoyongzhu
xiaoyongzhu previously approved these changes Nov 28, 2022
…ngly

Signed-off-by: Jun Ki Min <42475935+loomlike@users.noreply.github.com>
@xiaoyongzhu xiaoyongzhu merged commit 654d56e into feathr-ai:main Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Tag to execute build pipeline for a PR from forked repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants