Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call to Tilde e-Translation fails #14

Open
jnehring opened this issue Jun 8, 2016 · 5 comments
Open

Call to Tilde e-Translation fails #14

jnehring opened this issue Jun 8, 2016 · 5 comments

Comments

@jnehring
Copy link
Member

jnehring commented Jun 8, 2016

curl -X POST --header "Content-Type: text/html" --data "@SB50.html" "http://api-dev.freme-project.eu/current/e-translation/tilde?source-lang=en&target-lang=de"

SB50.txt

This might fix freme-project/e-Internationalization#35

@jnehring
Copy link
Member Author

jnehring commented Jun 8, 2016

The call fails at Tilde. This CURL

curl -X POST -d "@test.txt" --header "Content-Type: application/x-turtle" "https://services.tilde.com/translation/?sourceLang=en&targetLang=de"

using this test.txt

produces http status 500 (internal server error) and the message

{
    "Message": "Internal server error. Reason: Response status code does not indicate success: 400 (Error)."
}

@andish can you please take a look?

@andish
Copy link
Contributor

andish commented Jun 8, 2016

seems that there is an error on longer text for translation. I will invest this

@andish
Copy link
Contributor

andish commented Jun 9, 2016

The reason is because the sample contains a sentence that contains more than 200 words. In natural language texts such cases do not exists. But as it is copied from wikipedia and all text concatenated into single sentence, it leads to the error: "sentence too long".
This is sentence:

Coldplay Rocks Colorful Halftime Show With Help From Beyonce, Bruno Mars — ABC News (US), February 7, 2016 Broncos' defense dominates Panthers in Super Bowl 50 win — ESPN, February 7, 2016 Broncos defeat Panthers 24-10 to win Super Bowl 50 — WGN-TV, February 7, 2016 Share this:       Categories:  February 9, 2016 Published North America United States Culture and entertainment Sports Football (American) National Football League Hidden categories:  Pages with defaulting non-local links Pages with categorizable local links Navigation menu Personal tools Not logged in Talk Contributions Create account Log in Namespaces Article Collaboration Opinions  Variants  Views Read Edit View history More  Search     Navigation Main page Newsroom Recent changes Random article Archives Free use media upload Wikinews Help Pillars of writing Write an article Water cooler Style guide Live chat Donate Contact us Regions Africa Asia Central America Europe Middle East North America Oceania South America   Languages Add links Print/export Create a book Download as PDF Printable version Tools What links here Related changes Special pages Permanent link Page information Cite this page Shorten URL  This page was last modified on 9 February 2016, at 04:20

@jnehring
Copy link
Member Author

jnehring commented Jun 9, 2016

So I guess that the bug at least for this example will be fixed by freme-project/e-Internationalization#39 because it will introduce a lot of linebreaks and break the long sentence

@jnehring
Copy link
Member Author

We will not fix this on the FREME side but I leave the issue open so the problem is documented.

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

No branches or pull requests

2 participants