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

Updates to exported terms #16

Closed
31 tasks
gkellogg opened this issue Mar 22, 2023 · 8 comments · Fixed by #17
Closed
31 tasks

Updates to exported terms #16

gkellogg opened this issue Mar 22, 2023 · 8 comments · Fixed by #17
Assignees
Labels
spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2)

Comments

@gkellogg
Copy link
Member

Relevant to w3c/rdf-star-wg#37, I suggest we make the following changes to defined terms:

  • Change denote to reference denote in RDF12-CONCEPTS
    <dfn id="dfn-denote" data-cite="RDF12-CONCEPTS#dfn-denote" data-lt="denote" data-local-lt="denoted">denotes</dfn>
  • Change denotation to reference referent in RDF12-CONCEPTS
    <dfn id="dfn-denotation" data-cite="RDF12-CONCEPTS#dfn-referent">denotation</dfn>
  • Change entail to reference entailment in RDF12-CONCEPTS
    <dfn id="dfn-entail" data-cite="RDF12-CONCEPTS#dfn-entailment" data-lt="entail" data-local-lt="simple entailment|entailment">entails</dfn>
  • Change equivalent to reference equivalence in RDF12-CONCEPTS
    <dfn id="dfn-equivalent" data-cite="RDF12-CONCEPTS#dfn-equivalence">equivalent</dfn>
  • Change referent to reference referent in RDF12-CONCEPTS
    <dfn id="dfn-referent" data-cite="RDF12-CONCEPTS#dfn-referent">referent</dfn>

Make the following exported terms local (still referencable, but not exported to WebRef):

  • extension
    <dfn class="no-export lint-ignore">extension</dfn>
  • identify
    <dfn class="no-export lint-ignore" data-local-lt="identified">identify</dfn>
  • instance with respect to
    <dfn class="no-export lint-ignore">instance with respect to</dfn>
  • invalid
    <dfn class="no-export lint-ignore">invalid</dfn>
  • merging
    <dfn class="no-export lint-ignore">merging</dfn>
  • merge
    <dfn class="no-export lint-ignore">merge</dfn>
  • monotonic
    <dfn class="no-export lint-ignore">monotonic</dfn>
  • rdfs1
    <dfn class="no-export lint-ignore">rdfs1</dfn>
  • rdfs2
    <dfn class="no-export lint-ignore">rdfs2</dfn>
  • rdfs3
    <dfn class="no-export lint-ignore">rdfs3</dfn>
  • rdfs4a
    <dfn class="no-export lint-ignore">rdfs4a</dfn>
  • rdfs4b
    <dfn class="no-export lint-ignore">rdfs4b</dfn>
  • rdfs5
    <dfn class="no-export lint-ignore">rdfs5</dfn>
  • rdfs6
    <dfn class="no-export lint-ignore">rdfs6</dfn>
  • rdfs7
    <dfn class="no-export lint-ignore">rdfs7</dfn>
  • rdfs8
    <dfn class="no-export lint-ignore">rdfs8</dfn>
  • rdfs9
    <dfn class="no-export lint-ignore">rdfs9</dfn>
  • rdfs10
    <dfn class="no-export lint-ignore">rdfs10</dfn>
  • rdfs11
    <dfn class="no-export lint-ignore">rdfs11</dfn>
  • rdfs12
    <dfn class="no-export lint-ignore">rdfs12</dfn>
  • rdfs13
    <dfn class="no-export lint-ignore">rdfs13</dfn>
  • D-satisfiable
    <dfn class="no-export lint-ignore">D-satisfiable</dfn>
  • satisfiable recognizing D
    <dfn class="no-export lint-ignore">satisfiable recognizing D</dfn>
  • Skolemization
    <dfn class="no-export lint-ignore">Skolemization</dfn>
  • standardize
    <dfn class="no-export lint-ignore">standardize</dfn>
  • D-unsatisfiable
    <dfn class="no-export lint-ignore" data-lt="D-unsatisfiability">D-unsatisfiable</dfn>
@gkellogg gkellogg added the spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial label Mar 22, 2023
@pfps
Copy link
Contributor

pfps commented Mar 23, 2023

Is this an enhancement or is it just editorial?

@gkellogg
Copy link
Member Author

I suppose that's a matter of opinion. You might know better how important advertising these terms is outside of the spec itself. It should not break any existing links into the specification, but for the first five terms flowing the link will take you to RDF Concepts rather than the location of the "definition" in RDF semantics. Alternatively, these could be made internal ("no-export") terms like the others, but there should probably be text added to reference the exported term from RDF Concepts.

My purpose was to note that we have terms defined in parallel or that we have terms we might not really want to export.

@pfps
Copy link
Contributor

pfps commented Mar 23, 2023

I think that most or all of these are just editorial changes.

Is there a document that describes what the arguments to dfn mean?

@gkellogg
Copy link
Member Author

I can create a PR do handle this.

The ReSpec Wiki documents the attributes (for the most part), but there is a certain amount of experience and trial-and-error involved in figuring these out.

@gkellogg gkellogg self-assigned this Mar 23, 2023
@pfps
Copy link
Contributor

pfps commented Mar 23, 2023

That would be useful. Can you create a PR for the local-only ones by themselves? They all seem to be purely editorial and non-controversial.

I think it would be a good idea to collapse the terminology in semantics to use only denotes and referent and to check what the situation in concepts is. As long as no anchors are lost this would be a non-controversial editorial change. I'll put together a PR for this. Then a final PR can do the first five changes.

@pfps pfps added Editorial Errata management: this erratum is editorial spec:editorial Minor change in the specification (markup, typo, informative text; class 1 or 2) labels Mar 23, 2023
gkellogg added a commit that referenced this issue Mar 23, 2023
@gkellogg
Copy link
Member Author

That would be useful. Can you create a PR for the local-only ones by themselves? They all seem to be purely editorial and non-controversial.

I can split #17 between the "no-export" and the data-cite changes.

I think it would be a good idea to collapse the terminology in semantics to use only denotes and referent and to check what the situation in concepts is. As long as no anchors are lost this would be a non-controversial editorial change. I'll put together a PR for this. Then a final PR can do the first five changes.

If you add data-lt (or data-lt-local) to a definition, you can use that as the body of an anchor and it will automatically reference the single definition. For example, this is already the case for "simple interpretation" and "simply entail".

<dfn data-lt="simply entail">simple interpretation</dfn>

That could be done for denotes and refers to for example:

The words <dfn id="dfn-denote" data-cite="RDF12-CONCEPTS#dfn-denote" data-lt="denote|refers to" data-local-lt="denoted">denotes</dfn>
      and <a>refers to<a>

Although, in this case, they both reference denote in RDF Concepts. It does allow you to simply use <a>denote</a> and <a>refers to</a> in the body of the document.

@pfps
Copy link
Contributor

pfps commented Mar 23, 2023

No, leave #17 as is. I'll update my PR after #17 is merged in.

The problem with using both denotes and refers to is that people end up thinking that they are different, even with the disclaimer near the beginning. Using one term removes this problem. Also, refer is used in a non-technical sense in a few places in Semantics so it is a good idea to not use in a technical sense.

Of course, this is all just wording changes and thus editorial.

@gkellogg
Copy link
Member Author

gkellogg commented Mar 23, 2023

Note, to be particularly safe, it's possible that there are existing links into #dfn-refers-to, so a pattern I've used to preserve the entry point is to add <span id="dfn-refers-to"></span> near the definition of denote`. It's a pain to manage manually, but it makes sure that any inbound links for deprecated terms still go to the right place.

@pchampin pchampin removed the Editorial Errata management: this erratum is editorial label Mar 27, 2023
@pfps pfps removed the spec:enhancement Change to enhance the spec without affecting conformance (class 2) –see also spec:editorial label Mar 27, 2023
@pfps pfps closed this as completed in #17 Mar 31, 2023
pfps pushed a commit that referenced this issue Mar 31, 2023
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 a pull request may close this issue.

3 participants