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

So many zeros #46

Closed
jnehring opened this issue Oct 2, 2015 · 7 comments
Closed

So many zeros #46

jnehring opened this issue Oct 2, 2015 · 7 comments
Labels

Comments

@jnehring
Copy link
Collaborator

jnehring commented Oct 2, 2015

I have installed the new version of MT-ComparEval on a DFKI server. The new metrics show mostly zero. See example here or the screenshot:

image

This looks like a bug to me. I am not sure to whom to assign this issue.

@jnehring jnehring added the bug label Oct 2, 2015
@martinpopel
Copy link
Collaborator

I installed the new version yesterday and I see all metrics non-zero (for CS->EN batch2q Pilot0 and Pilot1). However, the import now takes 3 hours for one task! So we really need to fix #44 . Meanwhile, I will probably comment out the Hjerson metrics from the default config file.

I am not sure to whom to assign this issue.
Hjerson was added by @pompomon.

@jnehring:
Can you check if the Python script (for computing Hjerson, by Maja Popovic) has not died?
Can you replicate the problem on a small data set (10 sentences)?
Can you share this data set (upload it somewhere)?

@jnehring
Copy link
Collaborator Author

jnehring commented Oct 2, 2015

Can you check if the Python script (for computing Hjerson, by Maja Popovic) has not died?

I did not run a python script. After installation I only ran bin/watcher.sh which runs a PHP script. Maybe thats the problem. Can you point me to the python script?

Can you replicate the problem on a small data set (10 sentences)?
Can you share this data set (upload it somewhere)?

Will do.

@martinpopel
Copy link
Collaborator

The script should be run automatically by the watcher. See Hjerson.php. Maybe you can add some logging there (e.g. add 2> log$hashId.stderr | tee log$hashId.stdout | to the $externalCmd on line 11).

@jnehring
Copy link
Collaborator Author

I downloaded the latest sources and re-run the importer. Now the problem of this issue does not occur. The metrics that produced the zeros are disabled by configuration and the respective table cells are empty.

@martinpopel
Copy link
Collaborator

Yes, I disabled Hjerson in the default config because of #48 .

@jnehring: Can you improve Hjerson, so it can be integrated in a faster way? Can I assign you to #48 ?

@jnehring
Copy link
Collaborator Author

Can you improve Hjerson, so it can be integrated in a faster way? Can I assign you to #48 ?

Sorry I dont have time to implement new features. I am responsible for maintaining the MT CompareEval installation of DFKI and cannot take new duties.

@lefterav
Copy link
Collaborator

Closed as reported to be solved

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

No branches or pull requests

3 participants