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

improve-sst-reference #333

Closed
wants to merge 1 commit into from
Closed

improve-sst-reference #333

wants to merge 1 commit into from

Conversation

reschke
Copy link
Contributor

@reschke reschke commented Feb 22, 2017

No description provided.

@@ -73,9 +73,11 @@ informative:
title: "Structured Streams: A New Transport Abstraction"
author:
- ins: B. Ford
date: 2007-08
date: 2007-10
Copy link
Member

Choose a reason for hiding this comment

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

The paper contains a date of August 2007. Where did you find the October date?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

over here: http://www.sigcomm.org/node/2648 - I guess August was the conference date, and October was the publication date.

Copy link
Member

Choose a reason for hiding this comment

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

xml2rfc should be able to simply cite this paper by its DOI; see https://www.rfc-editor.org/pipermail/rfc-interest/2016-December/009805.html. (But I have no idea how to do this from kramdown.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That would give you: https://xml2rfc.tools.ietf.org/public/rfc/bibxml-doi/reference.DOI.10.1145/1282427.1282421.xml?anchor=SST

<reference anchor='SST' >
  <front>
    <title>Structured streams</title>
    <author initials="B." surname="Ford" fullname="Bryan Ford">
      <organization></organization>
    </author>
    <date year="2007" month="October"/>
  </front>
  <seriesInfo name="ACM SIGCOMM Computer Communication Review" value="Vol. 37, pp. 361"/>
  <seriesInfo name="DOI" value="10.1145/1282427.1282421"/>
</reference>

...which is cool, but, for instance, doesn't have the direct link to the actual paper.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at the kramdown-rfc2629 code, it doesn't look like it yet supports the bibxml7 directory. Here are the currently-supported references:

        "RFC" => "bibxml", "I-D" => "bibxml3", "W3C" => "bibxml4", "3GPP" => "bibxml5",
        "ANSI" => "bibxml2",
        "CCITT" => "bibxml2",
        "FIPS" => "bibxml2",
        "IANA" => "bibxml2",
        "IEEE" => "bibxml2",
        "ISO" => "bibxml2",
        "ITU" => "bibxml2",
        "NIST" => "bibxml2",
        "OASIS" => "bibxml2",
        "PKCS" => "bibxml2",

Copy link
Contributor

Choose a reason for hiding this comment

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

This is odd. This depends on how you write the ref. SIGCOMM was in Aug 2007, the current citation could say Proceedings of ACM SIGCOMM, August 2007. ACM CCR carries all SIGCOMM publications in the next issue, as a special issue, which must have come out in Oct 2007. I'd keep the Proceedings date, since that's what's usually used.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, if you cite it as "proceedings", you cite the conference date. If you cite it as published in CCR, you cite the issue publication date.

Copy link
Member

Choose a reason for hiding this comment

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

Inspired by this, I learned a new appreciation (read: loathing) for ruby in the course of adding DOI support to kramdown. It ended up consequential to solving a bunch of other problems.

See cabo/kramdown-rfc#35

@janaiyengar
Copy link
Contributor

What are we doing with this PR?

@janaiyengar janaiyengar mentioned this pull request Mar 10, 2017
@janaiyengar
Copy link
Contributor

I've fixed the build error and merged as PR #386 (removed the url since it was breaking the 80-col rule. The url is unnecessary anyways.)

@martinthomson
Copy link
Member

I've a PR open on kramdown-rfc2629 that will eventually make this redundant.

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.

5 participants