Skip to content

Correct tau-bench data attribution in NOTICE - #20

Open
Liang-Chun Tsai (ltsai-dev) wants to merge 2 commits into
mainfrom
ltsai-microsoft-fix-tau-bench-notice
Open

Correct tau-bench data attribution in NOTICE#20
Liang-Chun Tsai (ltsai-dev) wants to merge 2 commits into
mainfrom
ltsai-microsoft-fix-tau-bench-notice

Conversation

@ltsai-dev

@ltsai-dev Liang-Chun Tsai (ltsai-dev) commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

The NOTICE incorrectly described all tau-bench airline JSON data as copied verbatim. This corrects the attribution by distinguishing the modified files from the unchanged reservation data.

Changes

  • Describe flights.json and users.json as derived from tau-bench and modified for ThinkingBox.
  • Retain reservations.json as copied verbatim from tau-bench.
  • Include only the modified data files in Microsoft's documented changes.

Test plan

  • git diff --check

Related issues

N/A

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s third-party NOTICE to accurately attribute the tau-bench airline JSON data as derived and modified rather than copied verbatim, aligning the notice with the actual provenance of the data files.

Changes:

  • Reworded the tau-bench airline data attribution to state the JSON files are derived from tau-bench and modified for ThinkingBox.
  • Added an explicit bullet under “Modifications by Microsoft Corporation” documenting the data-file modifications.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.

3 participants