-
Notifications
You must be signed in to change notification settings - Fork 14
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 acknowledgements and prepare v0.7.0
release
#304
Conversation
@cr-xu, @AnEichler, @Hespe, @RemiLehe, @jp-ga: Please let me know if you have any funding strings or similar that you would like to have mentioned here. If you have nice proposals (e.g. examples from other repositories) for formatting the acknowledgements section, they are also welcome. For now I would list the authors, maybe participating institutions, and provide the funding strings. |
Just throwing some ideas here, maybe the contributors can be in a bullet list, with trailing links to their GitHub user names (if applicable), for example
|
This is the funding string from my side: |
For the current XFEL project, we used
before. |
I've put together what I think is a nice Acknowledgements section now. I know the logos don't render. The absolute paths are on purpose to fix image rendering on PyPI and should fix themselves automatically as soon as this PR is merged onto Let me know if anything is missing. If I don't get any comments, I will merge this as is on Friday before lunch Hamburg time. |
v0.7.0
release
…tah into 267-update-acknowledgements
Description
Updates the acknowledgements, adding new authors and funding notices.
Motivation and Context
The current acknowledgements do not reflect the work of new contributors who have joined the project. Addresses #267. Closes #165.
Types of changes
Checklist
flake8
(required).pytest
tests pass (required).pytest
on a machine with a CUDA GPU and made sure all tests pass (required).Note: We are using a maximum length of 88 characters per line.