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

Add placeholder for population dataset #132

Merged
merged 2 commits into from
Oct 15, 2024
Merged

Add placeholder for population dataset #132

merged 2 commits into from
Oct 15, 2024

Conversation

znichollscr
Copy link
Collaborator

@znichollscr znichollscr commented Oct 15, 2024

Description

Checklist

Please confirm that this pull request has done the following:

  • Documentation added (where applicable)
  • Changelog item added to changelog/

Copy link

No changes to the database between 'main' branch and cd766fd

@znichollscr znichollscr requested a review from durack1 October 15, 2024 18:50
@znichollscr
Copy link
Collaborator Author

@durack1 @vnaik60 not ideal to only have a placeholder, but at least this makes clear the current state of play

Copy link

No changes to the database between 'main' branch and 030582a

@durack1
Copy link
Contributor

durack1 commented Oct 15, 2024

@znichollscr agreed. It would be ideal if we had more details, like someone/institution that was willing to provide this, but agreed if it's anticipated, then best to recognize this and note the vagueness

@znichollscr znichollscr merged commit c5054b8 into main Oct 15, 2024
4 checks passed
@znichollscr znichollscr deleted the popn-dataset branch October 15, 2024 19:32
@vnaik60
Copy link
Collaborator

vnaik60 commented Oct 15, 2024

More info here https://office.wcrp-cmip.org/Products/Projects/Messages.aspx?prjID=33&id=327#comments (only accessible to the Forcings TT)

@znichollscr
Copy link
Collaborator Author

Thanks Vaishali. That link appears to be dead for me, can you copy the key info here?

@vnaik60
Copy link
Collaborator

vnaik60 commented Oct 17, 2024

Will do once onlyoffice is accessible.

@znichollscr
Copy link
Collaborator Author

Back live, I've got the comment now thanks. See #135

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.

3 participants