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

I run your code both on PTB and CTB dataset, but in CTB data LAS could not achieve the performence in paper #2

Open
Woooooody opened this issue Dec 5, 2017 · 4 comments

Comments

@Woooooody
Copy link

Could you offer the LAS result of PTB and CTB. thx

@jcyk
Copy link
Owner

jcyk commented Dec 6, 2017

Sorry, but I don't have the results.
Could you report your results here? Just the LAS of CTB is not good?

@Woooooody
Copy link
Author

Thank you for your response. At present, I got USA: 89.86, LAS: 84.09 on CTB. But in paper, this number is 89.30 and 88.23, respectively. I got more good result than paper on UAS, but LAS is not good enough. By the way, I got the same result on PTB3.03 both of UAS and LAS. All in all, thanks for your share.

@baoy-nlp
Copy link

baoy-nlp commented May 25, 2018

in the lib/data.py line 25 and line 29 ,it assumes that the root rel must be “root”.This is not necessary in some data sets, It may be "ROOT".

@jcyk
Copy link
Owner

jcyk commented May 26, 2018

@baoy-nlp wow, thank you!

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

No branches or pull requests

3 participants