

TITLE
Add warnings for incompatible generation parameters

COMMENTS
4

REACTIONS
+1: 0
-1: 0
laugh: 0
hooray: 0
heart: 0
rocket: 0
eyes: 0

BODY
# What does this PR do?
While testing various generation configurations, I got confused when beam_sample's outputs (i.e. `num_beams > 1, do_sample=True`) failed to change with different settings of `top_k, top_p, temperature`. Then I realized that in my call to `generate()`, `do_sample` was still set to false even though I was cycling through various settings of top_k, top_p and temperature. 

Generate (and its helper methods) already contain some parameter compatibility checks. This adds a few more checks:
- `num_beams` must be >= 1
- when `do_sample` is not set, warn the user if any of `top_p, top_k, temperature` are not None (since they will have no effect)
- if `num_beams` is 1 (no beam search), warn user if any of `early_stopping, length_penalty` are not None (since they will have no effect)
- if an invalid set of params `num_beams` and `do_sample` are passed, raise ValueError. Note that since we also add a check for `num_beams < 1`, this final value error will never be raised, but this prevents the `generate` function from falling off the end of the ifelse chain if something is altered in the future.


## Before submitting
- [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- [x ] Did you read the [contributor guideline](https://github.com/huggingface/transformers/blob/master/CONTRIBUTING.md#start-contributing-pull-requests),
      Pull Request section?
- [ ] Was this discussed/approved via a Github issue or the [forum](https://discuss.huggingface.co/)? 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](https://github.com/huggingface/transformers/tree/master/docs), and
      [here are tips on formatting docstrings](https://github.com/huggingface/transformers/tree/master/docs#writing-source-documentation).
No doc changes necessary
- [ ] Did you write any new necessary tests?
ran tests, but no new functionality created, just warning messages


## Who can review?
 Please tag fewer than 3 people.
 Text Generation: @patrickvonplaten 