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

License issue #146

Closed
devjunhong opened this issue Jul 2, 2024 · 1 comment
Closed

License issue #146

devjunhong opened this issue Jul 2, 2024 · 1 comment

Comments

@devjunhong
Copy link

devjunhong commented Jul 2, 2024

Dear tsroten,

Thanks a lot for your great work on pynlpir.

Today, I've seen a strange case when I ran the pynlpir update, it's still complaining about the license in calling the pynlpir.open() function.

Steps to reproduce

  1. pynlpir update
  2. python -c "import pynlpir; pynlpir.open()"
  3. You will get pynlpir.LicenseError: Your license appears to have expired. Try running "pynlpir update"

Investigation

  1. pynlpir update
    i. It returns the same LicenseError
  2. Copy the NLPIR.user file from the https://github.com/NLPIR-team/NLPIR/tree/master/License/license%20for%20a%20month/NLPIR-ICTCLAS%E5%88%86%E8%AF%8D%E7%B3%BB%E7%BB%9F%E6%8E%88%E6%9D%83
    i. It returns the same LicenseError
  3. cat pynlpir/Data/yyyymmdd.err
    i. "Not valid license or your license expired! Please feel free to contact [email protected]!"
  4. cat pynlpir/Data/yyyymmdd.log
    i. "Other thread is under initialization"

Please feel free to leave any comment about this.

@devjunhong
Copy link
Author

devjunhong commented Jul 16, 2024

Hi folks, it's been resolved. pynlpir package is working fine.

It seems a problem from the NLPIR repository. The license had been updated 3 days ago.

https://github.com/NLPIR-team/NLPIR/tree/master/License/license%20for%20a%20month/NLPIR-ICTCLAS%E5%88%86%E8%AF%8D%E7%B3%BB%E7%BB%9F%E6%8E%88%E6%9D%83

and now I can recognise if this happens later, we need to post here. https://github.com/NLPIR-team/NLPIR/issues
Those top two tickets are the license concerns.

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

1 participant