-
Notifications
You must be signed in to change notification settings - Fork 3
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
Minor improvements in documentation #5
Comments
You are correct regarding the name of the fields. Let me know if you could submit a PR for the documentation. Regarding the type of field, you are also correct. It was my intention to allow for a simple string and for a complex structure, but I impose to be a structure in the model. Both of them in my mind are valid options, although I tested only with complex structures. I will check the feasibility to allow fields as a simple string (example a simple abstract) or a complex structure (like the one that you provided as example) |
This was my first and probably naive approach:
But I have not tried yet to weight the datasets. By the way, which are the list of valid values for |
I would structure the item in the following way: group can be any string that you would like to use, as long as it is consistent in term of upper or lower case. |
Based on the latest post, I would structure your item as it follows: although doi will not add any added value. The group is an arbitrary string that you assign to group together a number of items. The group in the scoring is not related to the type shown in the portal. Document type is a field of the panosc document model. |
I am in the process of populating the scoring DB. It is very well documented but I found some minor pitfalls that could be easily improved.
The documentation says that the model has 3 fields:
I found some issues when I was playing with it:
_
. When I was trying toPOST /items
the documentation says that each item should match the structure defined in the model section however I only make it to work when I removed the_
. It means thatgroup
instead of_group
andfields
instead of_fields
. I saw thatid
is an alias of_id
panosc-search-scoring/app/models/items.py
Line 9 in a94ab1e
I would propose to leave only
id
at least in the documentation that sounds to me more coherent_fields
type: it is a dictionary even if the documentation says that it can contain just a stringNext item did not work for me:
and will raise the following error:
This worked:
The text was updated successfully, but these errors were encountered: