Skip to content

Add 'format' arg to get_result_df#885

Merged
xiaoyongzhu merged 3 commits intofeathr-ai:mainfrom
loomlike:jumin/backward_compat
Nov 29, 2022
Merged

Add 'format' arg to get_result_df#885
xiaoyongzhu merged 3 commits intofeathr-ai:mainfrom
loomlike:jumin/backward_compat

Conversation

@loomlike
Copy link
Copy Markdown
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