-
Notifications
You must be signed in to change notification settings - Fork 0
Permalink for ocean colour dataset - second Tier 3 #281
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
Comments
As usual, you must provide the Dataset ID, which is: |
I had to add the admonition box in the Quality assessment statement section. Make sure to include that next time you submit a notebook. Please ensure that the notebook name is correct. If you need to make any modifications (e.g., addressing the reviewer's comments), please share the new version of the notebook here, and I will provide a new permalink. Let me know once the notebook is accepted, and I will merge the PR into the main branch. |
Hi Mattia, where can I find the code to include the admonition box? Cheers, |
Download the template, and look at the markdown cell ```{admonition} These are the key outcomes of this assessment
:class: note
* Finding 1
* Finding 2
* Finding 3
* etc``` |
Thanks Mattia. Is it normal that I have the ⓘ symbol instead of the bell as in the previous NB? |
Yes, ⓘ is correct. |
Ok. |
Sorry, I forgot to specify the unit of trends in the table. |
Thanks Mattia. Is it normal that the preview and the permalink do not display the Table in the same way? It looks like the Table is displayed correctly in the Preview, but not in the permalink. |
Yes, the preview is what matters. |
I see. Thanks |
Hi Mattia. I have to rerun the NB by reducing the period analysed. Should we cache the data again, right? |
yes, the functions using |
Thank you |
Done! |
Thanks Mattia. |
Hi Mattia, I need a new permalink for this one: Cheers |
I forgot to correct a couple of typos. Here the right one: Cheers |
It looks like the server didn't save the NB correctly after fixing the typos. Could you please provide me with a new permalink? Sorry about that Thank you! |
Hi Mattia, I made a few changes in the text and I need a new permalink. Thank you |
Hi Mattia, Could you please provide a new permalink for this? Cheers, |
Hi Mattia, since Joao asked me for the "permalink ending in .html", could you please confirm whether the "Preview" link is the one for the Final Jupyter Book? If it is a different one, could you please provide it? Cheers, |
This is the preview URL: This will be the URL of the notebook once it is approved and ECMWF merges the PR (I just removed |
Hi Mattia, |
Yes. The first one should work. My guess is that Joao needs the first one, but I'm not 100% sure. We only have permalinks (.ipynb) and previews (.html). There's no "permalink ending in .html" as there's no version control on the html files. |
Hi Mattia. I need a new permalink for this one: Thanks, |
Hi Mattia, I need a new permalink for this NB: Thanks, |
Hi @chiaravol, We now have a strict checker for URLs. I'm not going to bore you with technical details, but I had to change all urls pointing to a journal with their equivalent DOI. For example, this Here is the permalink: https://github.com/ecmwf-projects/c3s2-eqc-quality-assessment/blob/e828c4e31fbb0a5d5c58847e6caef92d540386fd/Satellite_ECVs/Ocean/satellite_satellite-ocean-colour_completeness_q01.ipynb |
Ok, thanks. I will do it in my next NB. Cheers, |
Data Type
Satellite ECVs - Ocean
Assessment Category
completeness
Dataset Name
satellite-ocean-colour
Question Number
1
Workflow ID
eqctier3-2c71dfc2-e31d-46c1-9911-272e1da3e7c4
Zipped Notebook
ocean_color_monthly.ipynb.zip
Environment
Anything else we need to know?
Hi,
Could you please send me the permalink for this new NB?
Cheers,
Chiara
The text was updated successfully, but these errors were encountered: