-
Notifications
You must be signed in to change notification settings - Fork 41
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
draft GSOC 2020 post #123
draft GSOC 2020 post #123
Conversation
@richardjgowers @kain88-de can you please handle the rest of the review/fixing/merging/announcing? I won't have time to do this until Monday. Thanks! |
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.
Looks good, just a couple of questions.
Note: didn't try to preview the actual page, so I can't tell if all the links are working.
_posts/2020-02-22-gsoc2020.md
Outdated
</p> | ||
|
||
MDAnalysis has been accepted as an [organization][org] into [Google Summer of Code 2020][gsoc]. If you are interested in working with us | ||
this summer as a student read the advice and links below and write to us on the |
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.
This sentence needs a comma, can I suggest: "If you are interested in working with us as a student this summer, ...."?
_posts/2020-02-22-gsoc2020.md
Outdated
## Close an issue of MDAnalysis | ||
|
||
You must have *at least one commit in the development branch of | ||
MDAnalysis* in order to be eligible, i.e.. you must demonstrate that |
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.
Would it be worth telling potential candidates to do this as soon as possible in order to avoid delays with reviews etc.. ?
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.
Previewed a test build; all the links look good (excepting the application/student signup link, but I'm assuming that's because applications aren't live yet).
One minor comment, and otherwise looks good.
_posts/2020-02-22-gsoc2020.md
Outdated
## Close an issue of MDAnalysis | ||
|
||
You must have *at least one commit in the development branch of | ||
MDAnalysis* in order to be eligible, i.e.. you must demonstrate that |
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.
there's an extra . in the i.e.
Added the TNG proejct. I'll merge this later unless someone beats me to it. Whoever hits merge needs to tweet too |
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.
Nice. I would emphasize more that we want early contact with students. I may have time in the evening for more edits.
_posts/2020-02-22-gsoc2020.md
Outdated
Summer of Code 2020][gsoc]. | ||
|
||
[Apply] as soon as possible; the application window opens on March 16, | ||
2020 but you should start engaging with us sooner than that: read on. |
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.
Should engage with us as soon as possible.
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.
We should mention that early and continuously engaging with us is the best way to improve chances for a student. Especially since our main goal is to attract new core developers.
_posts/2020-02-22-gsoc2020.md
Outdated
part of the application process you must familiarize yourself with [Google | ||
Summer of Code 2020][gsoc]. | ||
|
||
[Apply] as soon as possible; the application window opens on March 16, |
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.
Apply once the application windows opens ...
_posts/2020-02-22-gsoc2020.md
Outdated
also appreciate if you write more tests or update/improve our | ||
documentation. | ||
|
||
We recommend that you don't leave this until the last minute to allow |
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.
That you start with this before you dive deep into your project. It will help to get to know MDAnalysis and understand how the proposed ideas fit into MDAnalysis.
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.
I made my changes. I'm good with the blog post
I found some minor typos. Is there a limit number of students working on MDAnalysis during this years GSOC? If yes we should mention this. |
There’ll be a limit based on final number of available mentors, how many
slots Google offers us and number of suitable students that apply. None of
these are currently known.
…On Mon, Feb 24, 2020 at 08:56, Philip Loche ***@***.***> wrote:
I found some minor typos.
Is there a limit number of students working on MDAnalysis during this
years GSOC? If yes we should mention this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#123?email_source=notifications&email_token=ACGSGB45QGLWCQV7CH3U7ADREODU7A5CNFSM4KYZ3NUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMXBCRQ#issuecomment-590221638>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACGSGBZXUGWWMO7M4ZTNEULREODU7ANCNFSM4KYZ3NUA>
.
|
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.
@lilyminium I corrected the grammar mistakes you found.
Thanks everyone! |
fix MDAnalysis/mdanalysis#122