-
Notifications
You must be signed in to change notification settings - Fork 6
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
Enhancements Oct 2023 #185
Conversation
Sharing some feedback here after testing more thoroughly:
|
@alanorth thanks for the feedback!
In the DEV AReS not all handles are added in the plugins, but here is a comparison between the public API and the explorer API:
Explorer API number of records for each prefix:
Note: The prefix 10947 is added to the plugins in the DEV server now.
The button is moved to the top to go with the other pages.
Here I followed the same approach as for the explorer dashboard.
This is strange and it is appearing in other dropdowns as well, it looks like an issue with the dropdown plugin, as it appears in the dropdowns having many options with long text. It is also happening here https://cgspace.cgiar.org/explorer/. Will investigate more.
Nice suggestion, it is applied!
The syntax is anything mentioned here https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-query-string-query.html and it is using all default values except for two parameters:
|
Thanks for the quick response. I see the changes are already live on the dev server. The sticky headers look great! And about the show/hide "^" I will look around the web and see how it is done elsewhere—something in my muscle memory feels backwards on that action. Regarding Altmetric, let me check when I harvest tonight to see how many pages there are. I am surprised that there are so many, it should be 100 per page or 1000? I also see you added the Minor feedback on consistent style:
Examples of inconsistencyModal dialog on plugin start (red start button): Pink commit button, plugin start/stop buttons in reverse logical order: |
This morning I was investigating the Altmetric harvesting to try to understand if it is correct or not. The Altmetric plugin shows 731 pages: I looked up one item that was previously missing an attention badge in AReS and found it is missing entirely! I looked in both the Handle and DOI filters but cannot find the item, despite it being present on DSpace 7 Test:
So something funny going on... |
|
Thanks for the layout changes! Looking really good now. I love the sticky header! As for the mysterious missing 10568/118261 handle, I swear it wasn't there when I checked. 🤔 Hmmm... So I'm wondering about the 700+ pages of Altmetric mentions. That's 7,000 items with attention scores, but previously we were hitting limits on the free API because we were trying to crawl more than 1000 pages or so. The numbers don't make sense to me and it seems like we will be missing some scores. Lastly, I'm thinking about a migration plan. It's good we can still harvest old DSpace instances so we can deploy the new code on a Sunday morning perhaps and you help me to make sure we get all the value mappings especially. Then we swap the indexes to the DSpace 7 sites once they are ready. |
We were hitting the limit which was 100 pages (X 100 items per page = 10K items), now we are getting 700 something (X 100 items per page = 70K items) but only ~19.5K are matched with the handles harvested from the repositories. Sure we can schedule a call and we do the migration together. |
Oh yes of course, my math was off. Good so we are confident we are capturing as many of our outputs with attention scores as possible. Let me think about the time schedule for the migration... talk soon. |
No description provided.