[bugfix] Fix the issue that will save the model card if push to hub is not applied in HiDream

[leisuzz]

What does this PR do?

The PR fixed the issue that save_model_card is automatically processed if push_to_hub is not called.
Now the function save_model_card is placed the same spot like other models

Before submitting

• This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
• Did you read the contributor guideline?
• Did you read our philosophy doc (important for complex PRs)?
• Was this discussed/approved via a GitHub issue or the forum? Please add a link to it if that's the case.
• Did you make sure to update the documentation with your changes? Here are the
  documentation guidelines, and
  here are tips on formatting docstrings.
• Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

[leisuzz]

@sayakpaul Please take look at this PR, thank you for your help :)

[linoytsaban]

Hi @leisuzz! this was actually done on purpose so that even if the model is not pushed to the hub, a model card is saved locally with an automatically generated inference snippet for easy usage.
did it error for you when --push_to_hub is not enabled?

[HuggingFaceDocBuilderDev]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

[leisuzz]

Hi @linoytsaban, I'm facing the error with the save_model_card function. I think to provide the flexible way to select save model_card and push_to_hub is better.

[leisuzz]

Hi @linoytsaban, I'm facing the error with the save_model_card function. I think to provide the flexible way to select save model_card and push_to_hub is better.

Because I'm running the model locally, and save_model_card will need external connect to the website. Which will cause error for users that need local training