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.

Sign up for GitHub

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

Jump to bottom

add paper to README.md #5

Merged
merged 2 commits into from
Jan 31, 2024
Merged

add paper to README.md #5

merged 2 commits into from
Jan 31, 2024

Conversation

hadyelsahar
Copy link
Contributor

Why ?

  • updating the readme to have the paper link and bib after uploading to arxiv
  • slight modification to the fig

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jan 31, 2024
@antoine-tran
Copy link
Contributor

great to have the fig in the asset. Could we remove the .png file also in this PR ?

@antoine-tran antoine-tran self-requested a review January 31, 2024 08:45
antoine-tran
antoine-tran approved these changes Jan 31, 2024
View reviewed changes
Copy link
Contributor

@antoine-tran antoine-tran left a comment

Choose a reason for hiding this comment

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

Update a few errors in README and delete the .png

@antoine-tran antoine-tran merged commit 6de82f8 into main Jan 31, 2024
6 checks passed
@antoine-tran antoine-tran deleted the hadyelsahar-patch-2 branch January 31, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoine-tran antoine-tran antoine-tran approved these changes

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hadyelsahar @antoine-tran @facebook-github-bot