search

citation-style-language / documentation

Citation Style Language documentation
http://citationstyles.org/
Creative Commons Attribution Share Alike 4.0 International
44 stars 21 forks source link

et-al-min has to be greater than et-al-use-first #141

Open georgd opened 3 years ago

georgd commented 3 years ago

Stumbling upon this twice in a PR today I couldn’t find this restraint neither in the schema (I wouldn’t know how to express it there) nor in the documentation.

bdarcus commented 3 years ago

Stumbling upon this twice in a PR today I couldn’t find this restraint neither in the schema (I wouldn’t know how to express it there)

Pretty sure you need to use schematron for that. The existing sch schema should give some ideas, if you'd like to give it a try.

georgd commented 3 years ago

Stumbling upon this twice in a PR today I couldn’t find this restraint neither in the schema (I wouldn’t know how to express it there)

Pretty sure you need to use schematron for that. The existing sch schema should give some ideas, if you'd like to give it a try.

I just looked into the existing files in the schema repo and couldn’t find a hint about that. But my knowledge of the whole schema thing is all but profound.

bdarcus commented 3 years ago

If other reviewers approve the substantive change, we can add the logic to the schema, which is here:

https://github.com/citation-style-language/schema/blob/master/schemas/styles/csl.sch

bdarcus commented 3 years ago

Please format this change as one-sentence-per-line rather than 80-characters-per-line.

This document isn't currently formatted that way.

I'll see if I can find a script to automate the conversion.

bwiernik commented 3 years ago

Yes, but we have started in a few places to add new additions that way to reduce later changes.

bdarcus commented 3 years ago

Yes, but we have started in a few places to add new additions that way to reduce later changes.

Ah.

I experimented a bit, and concluded a completely automated solution isn't practical.

This, OTOH, if you use Emacs (I imagine other editors may have similar), is awesome!

https://emacs.stackexchange.com/a/473

georgd commented 3 years ago

But that won't give you a line per sentence, would it?

I was almost through when I remarked I was in the wrong branch...

bdarcus commented 3 years ago

But that won't give you a line per sentence, would it?

Yes; the fill-paragraph command converts this:

One line per
sentence.

One line. Per sentence.

... to this:

One line per sentence.
One line.
Per sentence.
denismaier commented 3 years ago

Will this work with one space after a period as well? Or do you need two spaces? Will it convert "Mrs. Smith said ..." to:

Mrs. Smith said ... Or.: Mrs. Smith said ...

?

bdarcus commented 3 years ago

Will this work with one space after a period as well?

No, it's not very smart as is.

But I wouldn't run this on an entire file; more paragraph-by-paragraph, or region-by-region.

It still simplifies the process.

bwiernik commented 3 years ago

I might also try using RStudio's editor, which has a function to convert between 80 spaces/sentence. It seems to be fairly smart.

bwiernik commented 2 years ago

@georgd Could you submit this change as a PR to the schema repo?