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

Combine multiple parameters #11

Closed
regstuff opened this issue Feb 9, 2020 · 3 comments
Closed

Combine multiple parameters #11

regstuff opened this issue Feb 9, 2020 · 3 comments

Comments

@regstuff
Copy link

regstuff commented Feb 9, 2020

Is it possible to combine the ml & lc parameters. For eg.
words(ml='destroy', lc='relationship', max=5)

Right now this gives the same result as words(ml='destroy', max=5)
It ignores the lc portion.

@gmarmstrong
Copy link
Owner

For this particular example, the Datamuse API returns the same list for both queries:
https://api.datamuse.com/words?ml=destroy&max=5
https://api.datamuse.com/words?ml=destroy&lc=relationship&max=5

So, python-datamuse behaves correctly here. Do you know of another example?

@regstuff
Copy link
Author

Sorry, guess I wasn't clear. The results ARE the same.
I was wondering if they could be different. So for the second list, we would expect words with meanings like "destroy" which follow "relationship"

@gmarmstrong
Copy link
Owner

This library is just a wrapper for the Datamuse API, so its purpose is to provide a Python interface for using that API. I don't have any control over how the Datamuse service responds, and changing those results is outside the scope of this library. You can contact the Datamuse developers here (according to the "contact us" link on Datamuse's website). This way, the Java library, Node.js module, and any other programs that use the service can receive the same new features.

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