Skip to content

Hacks to demo possible CiTO support#33

Open
egonw wants to merge 3 commits intoopenjournals:mainfrom
egonw:enhancement/cito2
Open

Hacks to demo possible CiTO support#33
egonw wants to merge 3 commits intoopenjournals:mainfrom
egonw:enhancement/cito2

Conversation

@egonw
Copy link

@egonw egonw commented Mar 16, 2023 •
edited
Loading

  • with the existing apa.csl
  • with just the lua filter from BioHackRxiv

image

@egonw egonw closed this Mar 16, 2023
@egonw
Copy link
Author

egonw commented Mar 16, 2023

Sorry, forgot something :(

@egonw egonw reopened this Mar 16, 2023
@egonw egonw mentioned this pull request Mar 17, 2023
Closed
@egonw
Copy link
Author

egonw commented Mar 17, 2023

@tarleb, when/if you have time, your independent confirmation that I did it correctly would be highly appreciated.

@tarleb
Copy link
Collaborator

tarleb commented Mar 22, 2023

Hi, sorry for the wait.

LGTM! A conservative approach to making this information available would be to put it into a JSON-LD file that could be served from the paper website.

Maybe we could use git subtree to keep the script in sync with the implementation at pandoc-ext, but that might require some additional changes in Inara.

@egonw
Copy link
Author

egonw commented Nov 17, 2024 •
edited
Loading

Just a quick ping. I have had a very busy year, but this is still on my radar and todo list. I have been hacking a bit on BioHackrXiv and getting more familiar with the approaches. But it's all evening hour effort.

cthoyt
cthoyt reviewed Nov 17, 2024
View reviewed changes
scripts/entrypoint.sh
# Note that the output file must be defined in the format's defaults file.
/usr/local/bin/pandoc \
--data-dir="$OPENJOURNALS_PATH"/data \
--lua-filter="$OPENJOURNALS_PATH"/data/filters/extract-cito.lua \
Copy link
Contributor

@cthoyt cthoyt Nov 17, 2024

Choose a reason for hiding this comment

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

why not also put this in the pdf.yaml configuration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@cthoyt cthoyt cthoyt left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@egonw @tarleb @cthoyt