fairseq icon indicating copy to clipboard operation
fairseq copied to clipboard

Added + to connect error message

Open Pchatain opened this issue 2 years ago • 0 comments

Before submitting

  • [Typo] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
  • [Yes] Did you read the contributor guideline?
  • [No] Did you make sure to update the docs?
  • [No] Did you write any new necessary tests?

What does this PR do?

Fixes the error message for when a custom model is not recognized by the model registry. Previously, only the first string was printed, f"Could not infer model type from {cfg}.", and not the later strings which are useful for debugging. Adding the connector + allows all three strings to be printed together following a failed assertion.

PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

Pchatain avatar Aug 22 '22 20:08 Pchatain