compiler-research / compiler-research.github.io

7 stars 49 forks source link

New page added for Research Area "Automatic Differentiation" #163

Closed QuillPusher closed 4 months ago

QuillPusher commented 5 months ago
QuillPusher commented 5 months ago

@davidlange6 Requested changes have been made.

davidlange6 commented 5 months ago

I think my two meta comments remain -most importantly if we want a page that introduces AD we need a more expert team member to start its content. This can help get the right focus and more fundamental references/resources.

QuillPusher commented 5 months ago

@vgvassilev Can you please nominate a Subject Matter Expert that can update/re-write this content (as requested by David above)?

vaithak commented 4 months ago

Hi @QuillPusher, I can help with this.

QuillPusher commented 4 months ago

That would be great @vaithak . Please directly make any changes that seem suitable to you. All the source documents that I've used for this info are linked on the post.

P.s., I'm not sure how this works, I've invited you as a collaborator on my fork of the website (branch name is new_page_for_ad). Or you can edit the raw file and share with me (my contact is on the teams page), if that's simpler.

vaithak commented 4 months ago

Hi @QuillPusher, I have tried to improve the content; this may still need some formatting and grammatical fixes, though. cc: @davidlange6

QuillPusher commented 4 months ago

Thanks for the changes and the pointers @vaithak , I ran it through a grammar tool and added the mentioned images.

@davidlange6 This is ready for your review again 😅

vgvassilev commented 4 months ago

@vaithak, is that good to go?

vaithak commented 4 months ago

@vaithak, is that good to go?

Yup, looks good to me with your suggested changes.

QuillPusher commented 4 months ago

Thanks, @vgvassilev Changes incorporated