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

Create 2019-08-13-Ang-Li-Three-People-in-Turing.md #67

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

sparkler0323
Copy link
Contributor

No description provided.

Copy link
Contributor

@GeorgiaHCA GeorgiaHCA left a comment

Choose a reason for hiding this comment

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

Ang, this is excellent!! I really enjoyed reading it, and I hadn't realised there were all these fascinating people at the Turing. Your blog post helped me find out there was someone else you work with who has a background in autism research, which is cool to know!

Sorry for my very pernickety comments. They are very minor. Just correcting some grammar and spelling. The main one is when you use 'the' and 'a' or 'an' - in general, I'd use 'the' in front of any noun, when you expect the reader to know exactly what you are referring to (even if only by context), e.g. "the Turing Institute" or "the University of Cambridge", and "a" in front of any other noun beginning with a consonant, and "an" for ones beginning with a vowel. There are some exceptions, but this holds most of the time. English can be so totally erratic - your blog is actually very well written and clear so just a few minor points to make it perfect!

Well done ⭐ 🌟 👍

_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved

## Bo Wang

Bo Wang is a visiting researcher at Turing and work with Professor Terry Lyons based at Oxford.
Copy link
Contributor

Choose a reason for hiding this comment

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

Bo Wang is a visiting researcher at the Turing and works with Professor Terry Lyons based at Oxford.

## Bo Wang

Bo Wang is a visiting researcher at Turing and work with Professor Terry Lyons based at Oxford.
I knew him because he is one of the organizers for Data science for mental health, an interest group aiming
Copy link
Contributor

Choose a reason for hiding this comment

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

I know him

Copy link
Contributor

Choose a reason for hiding this comment

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

Just realise this isn't clear - I mean change "I knew him" to "I know him" (I don't actually know him, although he sounds interesting so I wish I did!)

Bo Wang is a visiting researcher at Turing and work with Professor Terry Lyons based at Oxford.
I knew him because he is one of the organizers for Data science for mental health, an interest group aiming
to lead to suitable clinical interventions for mental health by using multi-modal data. Bo is interested in
developing clinically useful application for mental health by his expert in deep learning, also he’s working
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend you change the following:
application - applications
expert - expertise
also he's working - new sentence. He's also working

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
developing clinically useful application for mental health by his expert in deep learning, also he’s working
developing a clinically useful application for mental health through his expert in deep learning. He’s also working

_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved
patients. His work is very meaningful and helps many depressive patients to start a new life without the risk of
suicide. However, some of the patients tend to responde more to this surgery, which motiviated Anujan to further
investigate this heterogeneity and thus pursue a PhD degree at University of Cambridge. He's project here is pretty
cool and aims to identify some depression patients who could be most likely to respond for this surgery and find the
Copy link
Contributor

@GeorgiaHCA GeorgiaHCA Aug 13, 2019

Choose a reason for hiding this comment

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

Suggested change
cool and aims to identify some depression patients who could be most likely to respond for this surgery and find the
cool and aims to identify the depression patients who would be most likely to respond to this surgery and find the

suicide. However, some of the patients tend to responde more to this surgery, which motiviated Anujan to further
investigate this heterogeneity and thus pursue a PhD degree at University of Cambridge. He's project here is pretty
cool and aims to identify some depression patients who could be most likely to respond for this surgery and find the
optimal surgery target for each individual, which involving neurosurgery, neuroimaging and machine learning. He also
Copy link
Contributor

@GeorgiaHCA GeorgiaHCA Aug 13, 2019

Choose a reason for hiding this comment

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

Suggested change
optimal surgery target for each individual, which involving neurosurgery, neuroimaging and machine learning. He also
optimal surgery target for each individual, involving neurosurgery, neuroimaging and machine learning. He also

investigate this heterogeneity and thus pursue a PhD degree at University of Cambridge. He's project here is pretty
cool and aims to identify some depression patients who could be most likely to respond for this surgery and find the
optimal surgery target for each individual, which involving neurosurgery, neuroimaging and machine learning. He also
believes that the curcuit of this surgery targets is a transdiagnostic biomarker, and thus could be useful to apply
Copy link
Contributor

@GeorgiaHCA GeorgiaHCA Aug 13, 2019

Choose a reason for hiding this comment

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

Suggested change
believes that the curcuit of this surgery targets is a transdiagnostic biomarker, and thus could be useful to apply
believes that the curcuit of this surgery targets is a transdiagnostic biomarker, and thus it could be useful to apply

cool and aims to identify some depression patients who could be most likely to respond for this surgery and find the
optimal surgery target for each individual, which involving neurosurgery, neuroimaging and machine learning. He also
believes that the curcuit of this surgery targets is a transdiagnostic biomarker, and thus could be useful to apply
this surgery for all patients who had a strong desire of suicide. I really love his ideas and hope he can move this
Copy link
Contributor

@GeorgiaHCA GeorgiaHCA Aug 13, 2019

Choose a reason for hiding this comment

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

Suggested change
this surgery for all patients who had a strong desire of suicide. I really love his ideas and hope he can move this
this surgery to all patients who have strong suicidal urges. I really love his ideas and hope he can move this


## Alex Campbell
Alex Campbell is an enrichment student at Turing and a PhD candidate at University of Cambridge. During Master
period, Alex used deep leanring technology in financial data mining and then he switched his main interest into
Copy link
Contributor

Choose a reason for hiding this comment

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

Change leanring to "learning"

@GeorgiaHCA GeorgiaHCA self-requested a review August 15, 2019 15:01
_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved
Bo Wang is a visiting researcher at Turing and work with Professor Terry Lyons based at Oxford.
I knew him because he is one of the organizers for Data science for mental health, an interest group aiming
to lead to suitable clinical interventions for mental health by using multi-modal data. Bo is interested in
developing clinically useful application for mental health by his expert in deep learning, also he’s working
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
developing clinically useful application for mental health by his expert in deep learning, also he’s working
developing a clinically useful application for mental health through his expert in deep learning. He’s also working

_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved
_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved
_posts/blog/2019-08-13-Ang-Li-Three-People-in-Turing.md Outdated Show resolved Hide resolved

Anujan Poologaindran was a neurosurgeon in Canada and worked on a magic surgery for treatment-resistant depression
patients. His work is very meaningful and helps many depressive patients to start a new life without the risk of
suicide. However, some of the patients tend to responde more to this surgery, which motiviated Anujan to further
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
suicide. However, some of the patients tend to responde more to this surgery, which motiviated Anujan to further
suicide. However, some of the patients tend to respond more to this surgery, which motiviated Anujan to further

@KirstieJane
Copy link
Member

@sparkler0323 - why are some of these reviews marked resolved when they aren't updated yet?

Please also update from master because I've fixed the Travis continuous integration and so it will now actually tell you something useful (rather than break all the time)!

Ping me in the slack channel when you've addressed @GeorgiaHCA's points and I'll have a read through ✨

sparkler0323 and others added 17 commits August 19, 2019 16:47
@sparkler0323
Copy link
Contributor Author

@KirstieJane So sorry, as I couldn't find out how to make an edit inside this pull request. Should I close this pull request and re-edit it in my Github?

@KirstieJane
Copy link
Member

Hi @sparkler0323 - if you make changes in your repository's master branch the pull request will automatically update ✨ Ping @GeorgiaHCA or @minnieho1115 in the slack channel if you're having any difficulties 😄

@KirstieJane KirstieJane mentioned this pull request Aug 20, 2019
@sparkler0323 sparkler0323 reopened this Aug 21, 2019
Copy link
Contributor Author

@sparkler0323 sparkler0323 left a comment

Choose a reason for hiding this comment

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

Thanks Georgia, your review is super helpful and I have fixed the grammatical mistakes based on them!

Copy link
Contributor Author

@sparkler0323 sparkler0323 left a comment

Choose a reason for hiding this comment

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

FIx them ~

Copy link
Contributor

@GeorgiaHCA GeorgiaHCA left a comment

Choose a reason for hiding this comment

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

Well done! 🚀

@KirstieJane
Copy link
Member

@sparkler0323 - I can only see a couple of sentences on separate lines - can you please go through the blog post and put them all on separate lines for me to review?

@KirstieJane
Copy link
Member

Hi @sparkler0323! I've fixed the broken links in the master branch - please can you update your branch so that CI passes?

@KirstieJane
Copy link
Member

Pinging again - I'd like to get this blog post merged - its been open for 2 months.

Let me know if you have any questions about updating the branch @sparkler0323.

@sparkler0323
Copy link
Contributor Author

Hi @KirstieJane, it will be great if you can help me the error. I have updated your master to my Github repository and I'm not sure why the merge fails. Thanks for your time!

@KirstieJane
Copy link
Member

Hi @sparkler0323! If you click on the "Details" link next to the Travis CI you can see what the error is - in this case its that it's a broken link in a different blog post - I can fix that 😄

Thank you! I'll review over the weekend ✨

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