- Notifications
You must be signed in to change notification settings - Fork524
Template for the IEEE Transaction Journals#97
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| $endif$ | ||
| % Pandoc header | ||
| $for(header-includes)$ | ||
| $header-includes$ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
You are includingheader-includes twice.
| \printbibliography$if(biblio-title)$[title=$biblio-title$]$endif$ | ||
| $endif$ | ||
| $for(include-after)$ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I didn't see the matchinginclude-before
| % Some very useful LaTeX packages include: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
As things stand now there really isn't a way for users to activate any of these classes. You could make these options at the format function level and then pass pandoc arguments along to conditionally enable them.
Could you also add a call to test the format here:https://github.com/rstudio/rticles/blob/master/tests/testthat/test_formats.R#L42-L48 |
Would it be okay if I contribute or submit the requested revisions to this PR? I would like to get the |
@coatless Yes, please. Go ahead. Due to work overload here at school, I had planned to do the revisions at the end of this month. |
@Emaasit: Could you give me write access to your fork? Or would you prefer if I PR'd against your fork and on merge those changes would appear here? |
@coatless I have added you as a collaborator to my fork. That gives read/write permissions. |
…eans up a few areas
Emaasit commentedJan 21, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
@coatless Thanks for those patches. Two more fixes are needed before it's perfect.
Regarding support for other articles formats, I think we should add only the |
Good catches by the way.
where Regarding supporting the |
espinielli commentedJun 8, 2017 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
This PR looks quite a mature contribution...is there any plans to merge it? |
Latex template can be found on the conference site athttp://www.ieee.org/conferences_events/conferences/publishing/templates.html