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

GENERAL PR GUIDELINES #47

Closed
ucalyptus opened this issue Jan 17, 2020 · 4 comments
Closed

GENERAL PR GUIDELINES #47

ucalyptus opened this issue Jan 17, 2020 · 4 comments

Comments

@ucalyptus
Copy link
Member

ucalyptus commented Jan 17, 2020

Before you make a PR

  • Make sure links mentioned in your md especially those of images work properly.
  • Check if md file is already there . In that case rather edit that file instead of uploading.
  • Add your name to README against the review. It's useless without your name.
  • Image Format
    <img src="./images/image-link-no-spaces.png" width="30" />
  • Update your forked repo with the upstream . Learn Here
  • Create a new branch named name-papername in your fork. All changes should happen here and master should stay untouched.
  • Do not make changes in master.
  • Learn how to create pull requests
  • Incase changes are requested on a PR, Close the pull request. Then make the changes in your PR and only then reopen the closed request.
  • In case you made some mistakes that do not have any solution, delete your fork and fork again.
    Make the above steps now.
  • Report me on the group chat if any of the links or pages in https://iem-computer-vision.github.io/ICCV19-Paper-Review/ have an error in links.
@shivishbrahma
Copy link
Contributor

shivishbrahma commented Jan 17, 2020

For image links use ./images/<your_image_name.extension>, never use absolute links, use relative links... Correcting @ucalyptus

@shivishbrahma
Copy link
Contributor

By the way finally got a better way to solve with equations...
Codecog Latex Editor
just type the equation in the editor in latex format
For Latex symbols visit here
use <img> tag

<img  class="eqn-inline" src="https://latex.codecogs.com/svg.latex?h(x;&space;m_f,&space;m_c)&space;=&space;c(f(x;&space;m_f);&space;m_c)">

Rendered equation is below

for inline equation use class "eqn-inline" and for outline equations class "eqn-outline"

For help, you can take help from my paper Drop to Adapt

@ucalyptus @Nibba2018 @Palspal98 @himanshuraj001 @Rajatava @soumya997 @shatadru99 @shivamdn9

@Nibba2018
Copy link
Member

  • Filenames having special characters will lead to weird command-line errors, specifically INVALID PATH Error. For e.g this filename had a semicolon(:) and was interfering with git CLI, thus preventing me from updating my fork.
D:\Github\ICCV19-Paper-Review>git merge upstream/master
error: invalid path 'Fine-Grained Segmentation Networks: Self-Supervised Segmentation for Improved Long-Term Visual Localization.md'
Updating 9fd7491..048f44d

@ucalyptus
Copy link
Member Author

* Filenames having special characters will lead to weird command-line errors, specifically `INVALID PATH` Error. For e.g this [filename](https://github.com/IEM-Computer-Vision/ICCV19-Paper-Review/commit/048f44d29e9ba57fac22676a828084a723dbabba) had a semicolon(:) and was interfering with git CLI, thus preventing me from updating my fork.
D:\Github\ICCV19-Paper-Review>git merge upstream/master
error: invalid path 'Fine-Grained Segmentation Networks: Self-Supervised Segmentation for Improved Long-Term Visual Localization.md'
Updating 9fd7491..048f44d

This we have to make sure that we do not copy the colon or semicolon while creating the filename

@ucalyptus ucalyptus added PENDING Deadline reached and removed NOTICE_BOARD PENDING Deadline reached labels Jan 20, 2020
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

No branches or pull requests

9 participants