-
Notifications
You must be signed in to change notification settings - Fork 5
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
obs4MIPs project configuration needs tweak #635
Comments
Thanks @durack1 I did not know that all the CMIP5-era data was deleted by now. So agreed lets get this cleaned up right and finalized before any push to the server. |
It would be useful to schedule a chat with @gleckler1 (and possibly @taylor13) to make sure we're not missing any additional opportunities to make search easier for a user |
Sounds good, a timeframe for releases would be useful for me anyway. |
@sashakames @durack1 @taylor13 Great to see the advancements to metagrid! I have three recommendations, the first 2 of which are specific to the obs4MIPs project: 1) remove CMIP5 and CMIP6 from obs4MIPs facets, 2) add a new obs4MIPs facet called "Quick links" with underlying searches to variable_id, source_id and frequency, and 3) enable a capability for some project specific information when first landing project via metagrid (discussed below). It would be very helpful to display project specific info about the data available via metagrid (typically more specific than the project website). This could be done via a "data info" button below the project website button that links to something like an obs4MIPs_MetaGrid_README.md Even better than a new browser tab (to this README after pushing the data info button) would be if by default when somebody first goes to the project on metagrid the contents of the README are shown in the large Home window where search results are found. |
@sashakames @downiec just looping around on this, is there a chance we can get these GUI tweaks prioritized in the coming month? |
Sure, I'll see what I can do on esgf-dev1 in the short-term. Peter wanted some additionally changes, too, correct? When he's back we can schedule something. |
A minimal initial solution is live on esgf-dev1.llnl.gov and I think we can do a production migration soon. |
@sashakames dev1 represents real improvement so its great that it can migrate soon. I think for obs4MIPs a next step would be an option for a second button (the first being to the project website). This second button, "Project data info" could be an external link. PG will prepare a link so that when the time comes the button can be prototyped. |
@sashakames @downiec can we address this issue soon?? |
I've updated the facets, and we can start work on the second link that is more specific to data information. For the time being users can hopefully get info from the obs4MIPs page linked currently. |
Hero, thanks @sashakames. It seems we have a garbled "Source Idinstitution ID" under identifiers, so would be good to catch that too - otherwise looks great to me! |
Thank you, there was a missing , between "" "" which Python concats rather than an error, tricky! |
@gleckler1 was there anything else to tweak, considering we have @sashakames engaged on this? |
Thank you for removing the CMIP5 and CMIP6 buttons Sasha! It looks like a carriage return is missing between Source_id and Institution_id (see attached).
With regards to the homepage button, I can appreciate that a 2nd button might take more work. But I’m wondering if there might be a simple compromise… instead of the “Obs4MIPs Website” button pointing to https://pcmdi.github.io/obs4MIPs/ could it point to https://pcmdi.github.io/obs4MIPs/dataOnESGF.html ? If that can be done, then I will update the info on that (dataOnESGF) link to provide more specific information about obs4MIPs data. Consistent with that, if not too complicated, changing the name on the button from “Obs4MIPs Website” to “Obs4MIPs Data Info” would be great but less critical.
Thanks again for these improvements, and all the best for 2025!
Peter
From: Sasha Ames ***@***.***>
Date: Tuesday, January 7, 2025 at 12:00 PM
To: esgf2-us/metagrid ***@***.***>
Cc: Gleckler, Peter John ***@***.***>, Mention ***@***.***>
Subject: Re: [esgf2-us/metagrid] obs4MIPs project configuration needs tweak (Issue #635)
I've updated the facets, and we can start work on the second link that is more specific to data information. For the time being users can hopefully get info from the obs4MIPs page linked currently.
—
Reply to this email directly, view it on GitHub<https://urldefense.us/v3/__https:/github.com/esgf2-us/metagrid/issues/635*issuecomment-2576120304__;Iw!!G2kpM7uM-TzIFchu!xtTmM7GsUZP3IYvXuHCmEIPjpmrkqytCSSsnXrh3DD_SHz-ApLLL0sRcHa5mYn4pOhRrVsYXctj_ToRzHPPjoa784O0$>, or unsubscribe<https://urldefense.us/v3/__https:/github.com/notifications/unsubscribe-auth/ABCXVLP772V7H6AK2PNXKHT2JQW6FAVCNFSM6AAAAABUYJKZHSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZWGEZDAMZQGQ__;!!G2kpM7uM-TzIFchu!xtTmM7GsUZP3IYvXuHCmEIPjpmrkqytCSSsnXrh3DD_SHz-ApLLL0sRcHa5mYn4pOhRrVsYXctj_ToRzHPPjok_IAH4$>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@gleckler1 are we good to close this issue? |
No. "Obs4MIPs Website" and link to it needs to be replaced with “Obs4MIPs Data Info” (see above for link). Seems to be hard to modify. |
@downiec What do you think about this request? An additional link next to the website for select projects. If a link is present in the config .json we display it, otherwise just the website link. |
Yes I can work on this next, shouldn't be too difficult, but it will require changes to the database/schema, to accept more than 1 project link because currently the project url comes from the database entry associated with the project |
Thanks for pointing that out @downiec! |
I'm getting a bit concerned about "scope creep" with having multiple links to be honest. But it would be fine to rename a single link to something more meaningful. |
We can for the time being, update the link and rename the button, easily enough. To add more than one link, we would need a feature update to make the project_url field more flexible (it's just a string right now), so that we can add in more than one url per project, and potentially specific button labels per url. |
Undersood two buttons may required additional special case work. Just changing the link from the homepage to https://pcmdi.github.io/obs4MIPs/dataOnESGF.html would meet the need. |
The link and button names have been updated as requested, the changes are currently on aims2.llnl.gov and are incorporated into our latest v1.3.1 release. |
Describe the bug
The metagrid configuration for the obs4MIPs project is a little confused - see https://aims2.llnl.gov/search?project=obs4MIPs - there is a spurious "CMIP5" facet which is empty, see screenshot below. Even the CMIP6 facet is a little mis-named.
To Reproduce
Browse to https://aims2.llnl.gov/search?project=obs4MIPs
@gleckler1 pinging you on this so we can get this cleared up
The text was updated successfully, but these errors were encountered: