-
Notifications
You must be signed in to change notification settings - Fork 520
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
Template for the IEEE Transaction Journals #97
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 including header-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 matching include-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
@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 |
This PR looks quite a mature contribution...is there any plans to merge it? |
Latex template can be found on the conference site at http://www.ieee.org/conferences_events/conferences/publishing/templates.html