-
Notifications
You must be signed in to change notification settings - Fork 25
minor changes in paper.md #195
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
Conversation
Co-authored-by: Trevor James Smith <[email protected]>
paper.md
Outdated
| The HDF Group have provided good documentation for the HDF5 format, but while there are communities of developers beyond those of the HDF Group, recent events suggest that given most of those developers and their existing funding are based in the USA, some spreading of risk would be desirable. | ||
| To that end, a pure Python code covering the core HDF5 features of interest to the target scientific community, which is relatively small and maintained by an international constituency provides some assurance that the community can maintain HDF5 access for the foreseeable future. | ||
| A pure Python code also makes it easier to develop scripts which can work around data and metadata corruption should they occur, and has the additional advantage of being able to be deployed in resource or operating-system constrained environments (such as on mobile). | ||
| The HDF Group have provided good documentation for the HDF5 format, but while there are communities of developers beyond those of the HDF Group, recent events suggest that given most of those developers and their existing funding are based in the USA, and some spreading of risk would be desirable. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the new "and" doesn't make sense here, if we need a new word, we should "so"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nah, even "so" doesn't make sense :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"and" removed in 112fd97
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, you are right - sorry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@valeriupredoi Please go ahead and merge this, but can you then fix the one word that I have highlighted (remove an "and") (I think it's easier to do this than request a change to the pull request for such a trivial change.
…add country to Astral Software Inc., both USA
|
attn: @bnlawrence - @danielskatz very many thanks for your suggestions! I have changed affiliation 8 accordingly, and also added a country for Astral Software Inc. (correct spelling too) in 6aa7b6a |
Description
Closes #issue_number
Before you get started
Checklist