Skip to content

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

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

Merged
merged 6 commits into from
May 3, 2024

Conversation

QuillPusher
Copy link
Contributor

  • also linked to research page

  • (SEO Optimized) this will help populate CR website in Google Search results for "Automatic Differentiation"

  • Descriptions taken from/verified from Presentations/Papers found on CR website. Rephrased for better SEO ranking (no duplication from original content on PDFs/Papers/Presentations)

- also linked to research page

- (SEO Optimized) this will help populate CR website in Google Search results for "Automatic Differentiation"

- Descriptions taken from/verified from Presentations/Papers found on CR website. Rephrased for better SEO ranking (no duplication from original content on PDFs/Papers/Presentations)
@QuillPusher
Copy link
Contributor Author

@davidlange6 Requested changes have been made.

@davidlange6
Copy link
Contributor

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
Copy link
Contributor Author

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

@vaithak
Copy link
Contributor

vaithak commented Apr 17, 2024

Hi @QuillPusher, I can help with this.

@QuillPusher
Copy link
Contributor Author

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.

the input nodes. For every node, it merges all paths which originated at that node.
It tracks how every node affects one output. Hence, it calculates derivative of a single
output with respect to all inputs simultaneously - the gradient.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe, we can add some images for forward and reverse mode AD, even the one on the Wikipedia page should be good enough.

@vaithak
Copy link
Contributor

vaithak commented Apr 25, 2024

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

@QuillPusher
Copy link
Contributor Author

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 😅

@QuillPusher QuillPusher mentioned this pull request Apr 27, 2024
12 tasks
@vgvassilev
Copy link
Contributor

@vaithak, is that good to go?

@vaithak
Copy link
Contributor

vaithak commented May 1, 2024

@vaithak, is that good to go?

Yup, looks good to me with your suggested changes.

@QuillPusher
Copy link
Contributor Author

Thanks, @vgvassilev
Changes incorporated

@vgvassilev vgvassilev merged commit 589bc49 into compiler-research:master May 3, 2024
1 check passed
@QuillPusher QuillPusher deleted the new_page_for_ad branch May 4, 2024 18:38
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.

4 participants