-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add EntryPoints compatible with the (currently in alpha development) PyTerrier Artifacts API #659
Comments
I.e., see a first test here: https://github.com/mam10eks/pyterrier-alpha/blob/main/tests/test_artifacts_from_tira.py |
If we have a pyterrier artifact |
@TheMrSheldon, @potthast: One thing that just came to my mind: when we implement things like this (
we could render the corresponding pages, i.e.,
where all the endpoints directly allow to browse, i.e., I think this could be a first use-case for the new V1 REST API. I.e., I think it would make sense to move the metadata on which approaches have been archived to Zenodo to the tira database (currently I had this in the code of the python client to remove dependencies to the live system, but data.tira.io will be statically hosted, so this should be no problem). And when we have this in the tira database, we can make the endpoint on accessing what is archived where public (as it is public anyway) and traverse this endpoint during the build of the static What do you think? This would also allow things like I think this would combine very well. |
On that note: if we use I think this would be a good idea because we already have random-document access in ChatNoir, especially for large corpora like the ClueWebs, hence it would allow us to reduce the size of the static part that we host in |
I would like this URL to be more like https://static.chatnoir.eu/?index=cw22&uuid=CWLafZMrWbCnXKvqA7IKZg or similar. |
Indeed, we can change the Url under which ChatNoir provides the random document access, for the proof of concept, we could stick with the existing URL for the moment I think. |
This is highly related to this issue: #594 |
The goal here would be to be compatible with this potentially upcoming pull request for PyTerrier: terrier-org/pyterrier#436
The text was updated successfully, but these errors were encountered: