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

REST Application #31

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

jerinphilip
Copy link

REST API using WSME Extension has been added using blueprints.
Currently mounted at /rest.
/rest -> gives a dict keyed by available modules
keys are list of functions available within modules

REST API using WSME Extension has been added using blueprints.
Currently mounted at `/rest`.
/rest -> gives a dict keyed by available modules
         keys are list of functions available within modules
@copyninja
Copy link
Member

It looks like you are ignoring CI results here. Please fix them

@jerinphilip
Copy link
Author

@copyninja: balasankarc's commits on spellchecker and indicstemmer should fix the issues. It's already been brought up to be due to rreplace -> setupfiles breaking upstream.

@copyninja
Copy link
Member

OK I will reschedule the build.

@copyninja
Copy link
Member

Rebuild still fails but it looks like some dependency issue from spellchecker.

Collecting libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4))

  Could not find a version that satisfies the requirement libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4)) (from versions: )

No matching distribution found for libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4))

@jerinphilip @balasankarc can you guys have a look collectively and sort this out?.

@balasankarc
Copy link
Member

@jerinphilip You probably have to fix requirements-modules.txt to point to new egg names. Like

git+https://github.com/libindic/indicngram.git#egg=libindic_ngram
git+https://github.com/libindic/indicstemmer.git#egg=libindic_stemmer

etc.

Also, isn't the libindic repo the latest one? Not Project-SILPA.

@jerinphilip
Copy link
Author

@balasankarc: Project-SILPA -> libindic is a rename. Not an archive. The link points to libindic.

@balasankarc
Copy link
Member

Ok, then make necessary changes in requirements.txt. :)

@jerinphilip
Copy link
Author

Oh, the egg names. Thanks. Is the change a result of the new namespace hierarchy?

@copyninja
Copy link
Member

I've requested a rebuild hope everything works fine now.

@jerinphilip
Copy link
Author

@balasankarc

Collecting libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4))

  Could not find a version that satisfies the requirement libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4)) (from versions: )

No matching distribution found for libindic_ngram (from libindic-spellchecker->-r requirements-modules.txt (line 4))

Can you look into this? It'll be easier if you put the correct eggs. Thanks.

@balasankarc
Copy link
Member

@jerinphilip Check the PR I made against your fork (Since it was a small change, I am made it against your fork and not the libindic one). Once you pull it in, it should show up here also.

@copyninja
Copy link
Member

Guys take a deep breath and sort the new error :-).. Please don't ping pong on the error.

@balasankarc
Copy link
Member

@copyninja :) . Sure.

@copyninja
Copy link
Member

@jerinphilip @balasankarc Looks like you guys still didn't fix this. Please have a look and correct it soon

@balasankarc
Copy link
Member

@copyninja Yeah. We are discussing about it and hopefully will fix it soon. I don't have much knowledge about WSME and am still getting my basics right.

I just relocated to a new place and was busy with all the shifting. Also, I haven't got any decent internet connectivity setup. I hope to get it all right by end of this week.

@Suman121
Copy link

Hi guys , i am trying to install these modules but getting same error. Is it fix???
PFA screen shot for your reference.
ngrammodulefailed

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

Successfully merging this pull request may close these issues.

4 participants