Skip to content

[AMORO-2895] Add more image related properties for the amoro optimizers#2896

Merged
zhoujinsong merged 3 commits intoapache:masterfrom
tcodehuber:issue-2895
Jun 12, 2024
Merged

[AMORO-2895] Add more image related properties for the amoro optimizers#2896
zhoujinsong merged 3 commits intoapache:masterfrom
tcodehuber:issue-2895

Conversation

@tcodehuber
Copy link
Copy Markdown
Contributor

Why are the changes needed?

Close #2895.

Brief change log

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@tcodehuber tcodehuber requested review from baiyangtx and czy006 June 7, 2024 01:43
Copy link
Copy Markdown
Member

@xleoken xleoken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

Copy link
Copy Markdown
Contributor

@czy006 czy006 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@baiyangtx baiyangtx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good job. Would you please add new properties to docs ?

@github-actions github-actions bot added the type:docs Improvements or additions to documentation label Jun 12, 2024
@tcodehuber
Copy link
Copy Markdown
Contributor Author

That's a good job. Would you please add new properties to docs ?

done.

@zhoujinsong zhoujinsong merged commit f964b59 into apache:master Jun 12, 2024
@zhoujinsong
Copy link
Copy Markdown
Contributor

Thanks for the work! @tcodehuber
Thanks for the review! @baiyangtx @xleoken @czy006

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement]: Add more image related properties for the amoro optimizers

5 participants