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 publications.md #320

Merged
merged 2 commits into from
Jan 9, 2024
Merged

Update publications.md #320

merged 2 commits into from
Jan 9, 2024

Conversation

aweerts
Copy link
Contributor

@aweerts aweerts commented Dec 18, 2023

added several pub;lications, added section Datasets etc

Issue addressed

Fixes #

Explanation

Explain how you addressed the bug/feature request, what choices you made and why.

Checklist

  • Updated tests or added new tests
  • Branch is up to date with master
  • Tests & pre-commit hooks pass
  • Updated documentation if needed
  • Updated changelog.md if needed

Additional Notes (optional)

Add any additional notes or information that may be helpful.

added several pub;lications, added section Datasets etc
@@ -164,8 +168,20 @@ Wang, X., Zhang, J., Babovic, V., 2016. Improving real-time forecasting of water
indicators with combination of process-based models and data assimilation technique.
Ecological Indicators 66, 428–439. <https://doi:10.1016/j.ecolind.2016.02.016>.

### Datasets
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the list of publications @aweerts . Nice to also list datasets. I would suggest to only list these if the dataset is not part of a report or paper (to minimize the work to keep track of these datasets and updating the list) , otherwise I think adding the report/paper is fine. Are/will these datasets be part of a publication?

@JoostBuitink , @dalmijn, @alimeshgi: what do you think, do you have a preference?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree to only list a dataset if it is not part of a publication.

Maybe it is nice to move the datasets to the bottom of this page, to put more emphasis on the reports?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes the datasetswill be part of publications (under review)
Hydrological Reference Dataset (Tier 1) for the ESA 4DHydro Open Science Catalogue
Droppers et al. 2024, submitted to Scientific Data

Added datasets will be part of publication, decided to to only list a dataset if it is not part of a publication.
Copy link
Contributor

@verseve verseve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update. As the listed datasets will be part of a publication, I have removed these.

@verseve verseve merged commit efbbd08 into Deltares:master Jan 9, 2024
10 of 11 checks passed
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.

4 participants