Skip to content
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

Update published source_id entries with rights; augment license #1070

Conversation

matthew-mizielinski
Copy link
Collaborator

Added blank license_info dictionaries to the 29 source ids that are Registered but not Published.

@matthew-mizielinski matthew-mizielinski marked this pull request as ready for review May 25, 2022 13:04
@matthew-mizielinski
Copy link
Collaborator Author

@durack1, I think his should make it a little easier in the future to update models that do publish data after registration (we'll have to do this for UKESM1-1-LL next) and it addresses the issue @MartinaSt mentioned on #1062

@durack1
Copy link
Member

durack1 commented May 25, 2022

@matthew-mizielinski this is one path forward, the other option was to contact the modelling groups to request/offer updates from these defaults, and at the same time query "Registered" modelling groups if they intend to publish - which would then allow deregistration in the case they want to do what you have with UKESM1-0-MMh (or even better publish) - a chat about the pluses and minuses may be useful

@durack1
Copy link
Member

durack1 commented May 26, 2022

Will leave this as pending, and can chat through the options with @MartinaSt during the WIP call next Tuesday - as changes have been made (UKESM1-1-LL is now published), this will need a refresh before merging

@durack1
Copy link
Member

durack1 commented Jun 9, 2022

@matthew-mizielinski this is now deprecated by #1028 (comment), and work underway (noted in link), so will close

@durack1 durack1 closed this Jun 9, 2022
@durack1 durack1 deleted the issue1062_matthew-mizielinski_add-blank-license-info-to-registered-models branch June 9, 2022 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants