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 citation button #1095

Closed
wants to merge 4 commits into from
Closed

Add citation button #1095

wants to merge 4 commits into from

Conversation

mjpost
Copy link
Member

@mjpost mjpost commented Nov 25, 2020

Adds a citation button per #412.

@mjpost
Copy link
Member Author

mjpost commented Nov 25, 2020

Here is what this looks like:

image

Clicking "Copy Citation" gives

[Improving Word Sense Disambiguation in Neural Machine Translation with Sense Embeddings](https://www.aclweb.org/anthology/W17-4702) (Rios Gonzales, Mascarell, & Sennrich, WMT 2017)

A note about how the venue acronym is computed: for 2020+, this is simple: the acronym can be looked up via the venue ID encoded in the paper's Anthology ID. Prior to 2020, we have two styles of IDs: those use "oldstyle letters" (which can similarly be looked up), and those using the "W" prefix. In that case, it's a bit trickier. I use the following heuristic. These events have the primary venue of the generic "WS", but often have joint associations with their real venue, e.g., WMT. So I look through all joint events, and return the acronym of the first such event that has no "oldstyle letter". This seems to work pretty well (including in the example).

One change I could make would be to omit the venue identifier entirely if it ends up as "WS", which is pretty meaningless.

We also do have some work cut out for us, associated some events that we were assigned "W" despite being long-running existing workshop or even conference venues.

@akoehn
Copy link
Member

akoehn commented Nov 25, 2020 via email

@mjpost
Copy link
Member Author

mjpost commented Nov 25, 2020

Maybe what I have produced is better referred to as a (extended) "reference" or "reference string" or "citation string".

I think the use for both is that they're simple to include in text documents, say if you're collecting a list of papers you want to read. My proposal was a bit more compact and relatively easy to produce.

@mjpost
Copy link
Member Author

mjpost commented Nov 25, 2020

How about this approach: a button at the top that just copies the summary text:

image

The icon formatting is off, I'd have to play with it. Here it is now:

<button type=button class="btn btn-clipboard btn-secondary btn-sm d-none" data-toggle=tooltip title="Copy as markdown" data-clipboard-text="[Simulated multiple reference training improves low-resource machine translation](https://www.aclweb.org/anthology/2020.emnlp-main.7) (Khayrallah et al., EMNLP 2020)">
  <i class="far fa-clipboard pr-2"></i>
</button>

@mbollmann
Copy link
Member

Re the formatting, remove this:

pr-2

This is Bootstrap-speak for padding right, size 2.


I wonder if this would be more intuitive as a text field that showed you the text it was copying, and that copies it when you click on it. (I've seen this many times on the web, but when I want to link to an example I can't find one, of course...)

@nschneid
Copy link
Contributor

Another option would be to provide a short citation field above "PDF:". For many purposes it's sufficient to copy directly off the page and paste as rich text (preserving links).

@nschneid
Copy link
Contributor

I.e.
image

@github-actions
Copy link

Build successful. You can preview it here: https://preview.aclanthology.org/markdown
This preview will be removed when the branch is merged.

@akoehn
Copy link
Member

akoehn commented Jun 14, 2021

The preview looks very broken -- no citation possibilities at all anymore.

@mjpost
Copy link
Member Author

mjpost commented Jun 14, 2021

This may be because we don't generate bib and other secondary files for previews. Maybe we should change that.

@mbollmann
Copy link
Member

Yeah, I was wondering about that. I also wanted to look into implementing the options we discussed in #412, but they'd probably also rely on the bibliography files being there.

@mjpost
Copy link
Member Author

mjpost commented Jun 14, 2021

Let's change it. Not at terminal but it's a quick fix to the "preview" target in the Makefile.

@mjpost
Copy link
Member Author

mjpost commented Aug 19, 2021

Superseded by (nicer work in) #1375.

@mjpost mjpost closed this Aug 19, 2021
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.

4 participants