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

Only first element of phrase used for Word Form Frequency calculation #44

Open
AgnesKim-oeaw opened this issue Jan 25, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@AgnesKim-oeaw
Copy link

see screenshot

grafik

If that problem cannot be solved or if you decide that it is not of interest to be able to calculate the frequency of all variants of all elements within the phrase deactivate that display dimension for phrase searches.

@ctot-nondef
Copy link
Member

@AgnesKim-oeaw thank you for reporting
@hpreki kannst du sagen ob das ein bug bei uns ist oder ob es das eine bestimmte syntax braucht im suchfeld

@hpreki hpreki added the bug Something isn't working label Jan 25, 2024
@hpreki
Copy link
Contributor

hpreki commented Jan 25, 2024

Es ist definitv ein bug. Die query wird nicht korrekt zusammengebaut.
Es fehlen die attribute ml1attr=word und ml1ctx=0<0~0>0
(cf. Swagger-doku )

Dafür sind ander überflüssige attribute drin wie usecorp=

Ein funktionierender, halbwegs sauberer call (aus der NoSke übernommen) wäre:

https://noskecrystal5corpsum.acdh-dev.oeaw.ac.at/run.cgi/freqml?corpname=amc4_demo&format=json&fmaxitems=5000&fpage=1&freq_sort=freq&group=0&showpoc=1&showreltt=1&showrel=1&freqlevel=1&ml1attr=word&ml1ctx=0<0~0>0&json={"concordance_query":[{"queryselector":"phraserow","phrase":"ein Haus","qmcase":false}]}

@ctot-nondef
Copy link
Member

the composable useWordFormFreqs needs to be revised, the parametrization seems to be copied from a different API call

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants