translate_text_with_glossary issue about deepl-python HOT 5 CLOSED

I finaly figured out.

As programming is just an hobby.. I don't have the reflex to look at the GitHub code, but I just figured out I could give it a shoot and most of the information is actually right in there.

For those who wonder:

def translate_text_with_glossary( self, text: Union[str, Iterable[str]], glossary: GlossaryInfo, target_lang: Union[str, Language, None] = None, **kwargs, ) -> Union[TextResult, List[TextResult]]:

We basicaly need to provide the full glossary object, not just the Id and if you are already calling without glossary, you can just call the basic function "translate_text". translate_text_with_glossary is actually calling translate_text.

from deepl-python.

Hello,

Sorry I was too quick. The "translate_text_with_glossary" function is still givin an error, but if you provide the glossary_id with the basic function "translate_text" it will work.

from deepl-python.

Hi Samuel,

I'm glad you figured out a solution. I'll add some additional explanation:

• GlossaryInfo objects are returned by the create_glossary, get_glossary, and list_glossaries functions. They contain the glossary ID string, and some details, for example the source and target language codes of the glossary.
• For translate_text_with_glossary(), the glossary parameter is required, and must be a GlossaryInfo object.
• For translate_text(), the glossary parameter is optional, and may be either a string (the glossary ID) or a GlossaryInfo object (from which the glossary ID is taken).
• While translate_text is general purpose, translate_text_with_glossary is intended as a wrapper/helper function.

The error you had is expected if you call translate_text_with_glossary with glossary set to a string. I'm sorry for the confusion, we can add a check to translate_text_with_glossary to catch this case in future.

Is your problem now resolved, or do you get another error?

from deepl-python.

Hello Daniel,

It did work using translate_text directly.
But after I looked at the code of translate_text_with_glossary, I tried to pass GlossaryInfo object and it still failed.

Best regards,
Samuel

from deepl-python.

Hi Samuel,

Okay, there shouldn't be an error when passing a GlossaryInfo to translate_text_with_glossary. Can you share your code and/or the error you get? Is it the same error as in your opening comment?

Regards,
Daniel

from deepl-python.