-
Notifications
You must be signed in to change notification settings - Fork 0
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
Consistency Assessment of Satellite Sea Surface Temperature for Climate Monitoring #197
Comments
I assume I can close #66? |
Also, I think something is not right. Why is the workflow ID of this notebook the same as #199? |
I copy pasted the workflows from the old issue, where the workflow was unique. We are not able to see the tasks because they went to the data provider after the splitting. Shall we re-open issues? In case yes, you can close all our SST issues, and we'll reopen them with the correct workflow id. Sorry for this oversight! |
Sorry, I'm a little lost. Do you have access to the unique workflow ID for each notebook? If you have it, you can just edit the workflow ID you provided in the form. Let me know when you are done, I don't get notifications for edits. |
Hi @vincenzodetoma, How would you like to proceed? Are you able to get the correct workflow ID from the CIM? |
Hi @malmans2, unfortunately still not. @dcianicnr83 asked for the reopening of all his tasks, but there were no answers yet. We'll come back to you as soon as we'll have something to work on. |
OK, if you need to speed up the process we can use the old ones and we'll fix it at the next round of review. |
Hi @malmans2, @dcianicnr83 we have modified the workflow id editing the issue opening form above. Thanks! |
I had to make a couple of fixes: You can only use H1 for the title, and one of the reference was not formatted correctly. I will share the permalink in a minute. |
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. |
All set. All notebooks had the same issues. Please start from the latest permalink if you are going to have another round of review. |
Dear @malmans2, with @dcianicnr83 we modified the notebook according to reviewer's suggestions. We ask thus for a new permalink. Thanks for your help and support! |
I found two issues in the notebook:
I fixed both of them, please start from the correct version if you'll need to revise the notebook. |
Data Type
Satellite ECVs - Ocean
Assessment Category
consistency
Dataset Name
satellite-sea-surface-temperature
Question Number
01
Workflow ID
eqctier3-8d67bc13-71c2-4aa1-b1bb-e7805302e724
Zipped Notebook
satellite_esacci_gmpe_monitor_q1_new.zip
Environment
Anything else we need to know?
Hi @malmans2, we (myself and @dcianicnr83) updated the notebook in #66 to match the new assessment categories and notebook template. Can you please provide us with a permalink and preview? Thanks
The text was updated successfully, but these errors were encountered: