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

add abe_2019_electro-oxidation_315 #44

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

nwoelfel
Copy link

@nwoelfel nwoelfel commented Dec 17, 2024

Depends on #61

  • Added a news file.

Literature Addition Checklist

For contributors and reviewers alike.

  • Name PR Add literature_identifier

Filename

  • Words and identifiers in the filenames must be lowercase.
  • The filename must be of type <Name>_<Year>_<FirstWordFirstPage>_<FirstPageNumber>_<fxy>_<Identifier>, where x is the figure number and y the subfigure label, i.e., 2b. For supporting information figures, use sfxy instead.
  • The figure number must match that of the figure in the article.

Bib file

  • The identifier must be identical with the filename as well as the citation key in the YAML.
  • Remove typos (or unwanted whitespace), specifically in the title such as Pt (111) to Pt(111).

SVG

  • The curve label should be short and unambiguous?
  • Figure label should be of type Figure: xy (not Figure: fxy).
  • The figure label must be identical to that in the article.
  • Comments should be complete sentences ending with a period.
  • A tags text label should be included in the SVG indicating the type of measurement (BCV, ORR, COOR, etc).
  • The units on the axis should be identical to those in the figure.
  • Considered a possible scaling factor?

YAML

  • The identifier must be identical to that in the filename.
  • Comments should be complete sentences ending with a period.

@nwoelfel nwoelfel marked this pull request as ready for review January 8, 2025 12:42
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.

2 participants