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

#1400: Unlimited retrieval and 500 glosses per page on Split Senses #1404

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

susanodd
Copy link
Collaborator

@susanodd susanodd commented Nov 28, 2024

This change is trivial.

  • @Woseseltops this is on signbank-susan now.

  • You do a Search on the dataset (NGT since that's what we're testing)

  • Then go to Analysis -> Split Up Senses (it shows them for the most recent query results)

  • The "browser debug window" has trouble showing all the nodes!!!!

  • But it does indeed work!!!

  • But I suspect it might not be showing all the results????

  • In the python code, I set it to display 500 results per page.

  • But there are only 2 pages shown.

  • Can you experiment with this yourself to see what the browser is doing? I'm using Firefox so I can't test it for Windows

  • I also don't know whether Firefox is just throwing stuff away because it's too much???

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant