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

lxsuite_key is required #42

Open
martinpopel opened this issue Nov 27, 2015 · 4 comments
Open

lxsuite_key is required #42

martinpopel opened this issue Nov 27, 2015 · 4 comments
Assignees

Comments

@martinpopel
Copy link
Member

perl -MTreex::Tool::LXSuite::LXConjugator -E '
my $conjugator = Treex::Tool::LXSuite::LXConjugator->new();'

produces an error Attribute (lxsuite_key) is required because the Client needs the key.
Note that I have created ~/.lxsuite2 file. Also there should be a warning with an explanation message if this file is missing.
The module above is needed by T2T::EN2PT::TurnVerbLemmaToAdjectives.

@joaoantonioverdade
Copy link
Contributor

It was using the old LXSuite interface.
It should be fixed now.

@martinpopel
Copy link
Member Author

I still get the same error (with the newest master).

When closing a ticket it is a good custom to mention the commit/pull request which fixes the issue (or use https://help.github.com/articles/closing-issues-via-commit-messages/).

@martinpopel martinpopel reopened this Mar 14, 2016
@joaoantonioverdade
Copy link
Contributor

I am not following.
Luis changed the interface, that way of calling the LXConjugator is no longer in use.
I changed it in the TurnVerbLemmaToAdjectives, so where and why are you using the old interface?
Did I miss any block?

@martinpopel
Copy link
Member Author

perl -MTreex::Block::W2A::PT::TokenizeAndTag -E 'my $b=Treex::Block::W2A::PT::TokenizeAndTag->new();'
results in error Attribute (lxsuite_key) is required at...
The block W2A::PT::TokenizeAndTag is still in use in Scen::Analysis::PT, which is in use in pt2en translation.

If Treex::Tool::LXSuite::LXConjugator is no longer in use and it is not planned to be fixed, I would suggest to remove it (as well as Treex::Tool::LXSuite::LXConjugator which depends on it) from the master branch.

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

2 participants