Skip to content

Some improved language, punctuation, and markup in rdf-semantics #126

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

TallTed
Copy link
Member

@TallTed TallTed commented Mar 27, 2025

it may be necessary to <dfn class="no-export lint-ignore">standardize</dfn> apart the blank node identifiers
by replacing them by others which do not occur in the other document(s).
For example, the two graphs represented by the following texts:</p>
it may be necessary to <dfn class="no-export lint-ignore">standardize</dfn> the blank node identifiers
Copy link
Contributor

@doerthe doerthe Mar 28, 2025

Choose a reason for hiding this comment

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

"standardize apart" is a fixed term (Logic Programming) and I would therefore not change that

Copy link
Member Author

Choose a reason for hiding this comment

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

How about this?

Suggested change
it may be necessary to <dfn class="no-export lint-ignore">standardize</dfn> the blank node identifiers
it may be necessary to <dfn class="no-export lint-ignore">standardize apart</dfn> the blank node identifiers

Copy link
Contributor

@afs afs Mar 28, 2025

Choose a reason for hiding this comment

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

RDF Semantics is not defining "standardize apart".

Just remove the dfn markup to leave the text:

"it may be necessary to standardize apart the blank node identifiers"

Copy link
Member Author

Choose a reason for hiding this comment

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

I think @afs advises well (hence part of this text suggestion), but based on what I found in documentation Google showed me when I asked "what does "Standardize apart" mean?", I've reordered some words, which I think makes the sentence more readable. (We might consider adding an external link to some Logic Programming text which goes further into this subject, which I believe we have not made a prerequisite for readers of our specifications.)

Suggested change
it may be necessary to <dfn class="no-export lint-ignore">standardize</dfn> the blank node identifiers
it may be necessary to standardize the blank node identifiers apart,

Copy link
Contributor

Choose a reason for hiding this comment

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

The operation name is "standardize apart" as @doerthe notes.
The google hits has it as the two words together each time.

For better or worse, the RDF Semantics introduction starts as the document goes on:
"This document defines a model-theoretic semantics ..."

(and why I haven't done a technical review of RDF Semantics.)

Copy link
Member Author

Choose a reason for hiding this comment

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

Any objection to this, which makes plain that it's a complex verb? I have spent some fruitless time searching for a page that clearly and simply defines this "operation", but all pages I've found treat it as preexisting knowledge, which is clearly a problematic assumption, as it was not preexisting for me. I would much rather entirely remove this term of art, and say the same thing with other words.

Suggested change
it may be necessary to <dfn class="no-export lint-ignore">standardize</dfn> the blank node identifiers
it may be necessary to "standardize apart" the blank node identifiers

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it is OK but (1) I am not an editor of this document (2) experts in the field may have a view.

Copy link
Member Author

Choose a reason for hiding this comment

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

ping oh ye experts in the field (@doerthe @franconi @niklasl (are there other such experts among us?))! Please chime in on my latest suggested change?

Copy link
Contributor

Choose a reason for hiding this comment

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

ping oh ye experts in the field (@doerthe @franconi @niklasl (are there other such experts among us?))! Please chime in on my latest suggested change?

I think it is ok, as it uses the appropriate term.

@pfps
Copy link
Contributor

pfps commented Apr 3, 2025

I don't see any need to add m-dashes around Examples 1 and 2. In fact, my view is that they are a distraction.

Although I prefer Oxford commas there is no reason to change to them in one place unless this is the only place where there is a missing Oxford comma.

@pfps pfps added the spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2) label Apr 3, 2025
@TallTed
Copy link
Member Author

TallTed commented Apr 4, 2025

I don't see any need to add m-dashes around Examples 1 and 2. In fact, my view is that they are a distraction.

They serve as a connector between the sentence fragments preceding and following the code blocks, and those code blocks. Their presence makes clear that there was no accidental omission of other punctuation (whether period, comma, or otherwise) or text. It is far more distracting to try to figure out what punctuation or text might have been left out, than it is to read the flow with the em-dashes.

Although I prefer Oxford commas there is no reason to change to them in one place unless this is the only place where there is a missing Oxford comma.

So if there are 2 missing terminal commas, in a document which otherwise has them in all eligible positions, there's no reason to add them in those 2 locations? What if there are 3 missing? Why should one missing comma be added, but two or more missing commas be left that way?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants