AlignmentResearch / KataGoVisualizer

MIT License
3 stars 1 forks source link

Update links and reorder logos #58

Closed AdamGleave closed 1 year ago

AdamGleave commented 1 year ago

Moved FAR logo first to reflect majority of authors now have FAR not CHAI affiliation.

Link directly to arXiv paper, remove our local PDF copy (be more DRY).

tomtseng commented 1 year ago

src/content.ts also links to /pdfs/go_attack_paper.pdf in a few places. It links to some specific sections of the paper (e.g., it links to appendix E.2) whose numberings have changed between the current arXiv version vs. the more up-to-date /pdfs/go_attack_paper.pdf, so we'll want to update the arXiv version before deleting /pdfs/go_attack_paper.pdf

AdamGleave commented 1 year ago

Ah thanks for catching the arXiv vs PDF discrepancy, I didn't realize the PDF was more up to date. Yeah, let's aim to update arXiv release soon and put this PR on hold until then.

netlify[bot] commented 1 year ago

Deploy Preview for goattack canceled.

Name Link
Latest commit e4a7a07747ea193e5d282dc2bcd36e478e7ea61a
Latest deploy log https://app.netlify.com/sites/goattack/deploys/64018653918fc200086e0e98
ed1d1a8d commented 1 year ago

Went ahead and just made some changes myself. Links are now updated and logos are smaller.

AdamGleave commented 1 year ago

Thanks for pushing this over the finish line, sorry I went AWOL on this PR.