Also agree. Can't recall ever reading about a default, don't think
it's a good idea, and it's not the intent of the current spec to allow
setting only one of the two et-al parameters of a set ("...-min" and
"...-use-first" for "et-al-..." and "et-al-subsequent-...").With the hierarchical name attributes, it's not entirely straightforward to test for the presence of both with validation, but otherwise we'd screen for it. Maybe we should require that both attributes of a set should occur on the same element. That would also make it easier to ascertain that the "...-min" attribute has a higher value than the "...-use-first" attribute, which is commonly violated in submissions. Rintze On Mon, Sep 5, 2016 at 9:58 AM, Sebastian Karcher <karc...@u.northwestern.edu> wrote: > Just to confirm that I agree the default shouldn't be there (and removing it > will make reviewing a tiny bio easier for Rintze and me. Thanks! > > Sent from my phone > > > On Sep 5, 2016 6:18 AM, <fo...@boss-reus.nl> wrote: >> >> >> CiteProc.NET also uses the default value of 1, probably because of the >> test at hand. I'll remove the default value too. >> >> Fouke >> >> >> Frank Bennett schreef op 2016-09-05 11:16: >> > Sylvester, >> > >> > Great. I've removed the constraint in the current processor release, >> > but it might be restored after discussion. From a quick check of the >> > repo, it looks like five styles rely on the default value. If that's >> > roughly representative of styles in the wild, the impact will be >> > small, but some people might be bitten. >> > >> > Two additional styles use an extremely high value for et-al-min, >> > presumably with the intention of listing all authors; those would not >> > be (much) affected. >> > >> > Curious to hear the views of @adam42smith and @rmzelle on the issue. >> > >> > Here is the list of affected styles: >> > >> > harvard-the-university-of-melbourne >> > ithaque >> > le-tapuscrit-note >> > philosophiques >> > geochimica-et-cosmochimica-acta (et-al-min="1000") >> > harvard-oxford-brookes-university-faculty-of-health-and-life-sciences >> > (et-al-min="100") >> > universite-libre-de-bruxelles-histoire >> > >> > Frank >> > >> > >> > >> > On Mon, Sep 5, 2016 at 5:02 PM, Sylvester Keil <sylves...@keil.or.at> >> > wrote: >> >> Hi Frank, >> >> >> >> Just to let you know, the test in question passes in citeproc-ruby >> >> too. >> >> After some digging I found the commit that sets the default to 1 to >> >> refer to this being citeproc-js' default value specifically (likely >> >> inferred from the test at hand). If you remove the default value, I'll >> >> also remove it from citeproc-ruby. >> >> >> >> Sylvester >> >> >> >> >> >> >> >> On Sun, 2016-09-04 at 14:51 +0900, Frank Bennett wrote: >> >>> This weekend, I'm overhauling some of the names code in citeproc-js, >> >>> to fix some long-standing bugs in attribute inheritance. One of the >> >>> test failures doesn't actually look like a failure, and I'd like to >> >>> confirm that I'm not misreading the spec. >> >>> >> >>> The test is this one: >> >>> >> >>> https://github.com/citation-style-language/test-suite/blob/master >> >>> /processor- >> >>> tests/humans/nameattr_EtAlUseFirstOnCitationInBibliography.txt >> >>> >> >>> In the fixture, a value for et-al-min is set on separate cs:name >> >>> nodes >> >>> inside cs:citation and cs:bibliography, and a value of 2 for >> >>> et-al-use-first is set on the cs:citation node. The test renders the >> >>> bibliography. >> >>> >> >>> The bibliography comes out with all three names given in the input. >> >>> The fixture RESULT has one name, truncated by et al. >> >>> >> >>> I don't see anything in the spec about defaulting to a value of 1 for >> >>> et-al-use-first, and it says that "[u]se of these two attributes >> >>> enables et-al abbreviation." Since the test provides a value for only >> >>> one of the attributes inside cs:bibliography, it sounds like listing >> >>> all three names there is the right thing to do (and the original >> >>> RESULT string was wrong). >> >>> >> >>> So ... should I amend the test there? >> >>> >> >>> Frank >> >>> >> >>> ------------------------------------------------------------------- >> >>> ----------- >> >>> _______________________________________________ >> >>> xbiblio-devel mailing list >> >>> xbiblio-devel@lists.sourceforge.net >> >>> https://lists.sourceforge.net/lists/listinfo/xbiblio-devel >> >> >> >> ------------------------------------------------------------------------------ >> >> >> >> _______________________________________________ >> >> xbiblio-devel mailing list >> >> xbiblio-devel@lists.sourceforge.net >> >> https://lists.sourceforge.net/lists/listinfo/xbiblio-devel >> >> >> > >> > >> > ------------------------------------------------------------------------------ >> > _______________________________________________ >> > xbiblio-devel mailing list >> > xbiblio-devel@lists.sourceforge.net >> > https://lists.sourceforge.net/lists/listinfo/xbiblio-devel >> >> >> >> ------------------------------------------------------------------------------ >> _______________________________________________ >> xbiblio-devel mailing list >> xbiblio-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/xbiblio-devel > > > ------------------------------------------------------------------------------ > > _______________________________________________ > xbiblio-devel mailing list > xbiblio-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/xbiblio-devel > ------------------------------------------------------------------------------ _______________________________________________ xbiblio-devel mailing list xbiblio-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/xbiblio-devel
