Skip to content

Add precision property to result and reportingDescriptor#766

Open
adityasharad wants to merge 1 commit intoeditor-revision-2026-03-05from
adityasharad/precision-property
Open

Add precision property to result and reportingDescriptor#766
adityasharad wants to merge 1 commit intoeditor-revision-2026-03-05from
adityasharad/precision-property

Conversation

@adityasharad
Copy link
Copy Markdown
Contributor

@adityasharad adityasharad commented Mar 5, 2026

Closes #611.
Proposal for SARIF 2.2.

Open questions for review:

  • Should this be in reportingConfiguration too? It doesn't seem like a property of the tool runtime, so I left it out.
  • Should we specify -1.0 as the default value like rank? That seemed odd to me, so I kept the default as absent/omitted.

@sthagen
Copy link
Copy Markdown
Contributor

sthagen commented Mar 5, 2026

@Torc-Charles-Wilson please kindly review.

@adityasharad
Copy link
Copy Markdown
Contributor Author

Notes from meeting

  • mention 'floating point' numbers
  • encourage limiting precision of decimals
  • mention that it's allowed to change between versions of a tool/component

@sthagen
Copy link
Copy Markdown
Contributor

sthagen commented Mar 6, 2026

My feedback on:

Open questions for review:

  • Should this be in reportingConfiguration too? It doesn't seem like a property of the tool runtime, so I left it out.

Seconded.

  • Should we specify -1.0 as the default value like rank? That seemed odd to me, so I kept the default as absent/omitted.

Seconded.

@sthagen sthagen mentioned this pull request Mar 7, 2026
8 tasks
@sthagen sthagen changed the base branch from main to editor-revision-2026-03-05 March 7, 2026 14:22
@sthagen
Copy link
Copy Markdown
Contributor

sthagen commented Mar 7, 2026

Changed target to branch oasis-tcs/sarif-spec/editor-revision-2026-03-05 as usual and discussed during the TC meeting 2026-03-05.

@sthagen
Copy link
Copy Markdown
Contributor

sthagen commented Mar 7, 2026

We need to merge PR # 767 first into this source branch and then both change-sets combined enter the oasis-tcs/sarif-spec/editor-revision-2026-03-05 branch.

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.

SARIF 2.2 proposal: precision field for reportingDescriptors and results

2 participants