Skip to content

docs: add NOTICE file for Opticks attribution#328

Open
plexoos wants to merge 2 commits into
mainfrom
add-notice
Open

docs: add NOTICE file for Opticks attribution#328
plexoos wants to merge 2 commits into
mainfrom
add-notice

Conversation

@plexoos
Copy link
Copy Markdown
Member

@plexoos plexoos commented May 14, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 14, 2026 14:38
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fabb8d5393

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread NOTICE.txt
This product includes software originally developed as part of the Opticks project:
http://simoncblyth.bitbucket.io/opticks/

This software is licensed under the Apache License, Version 2.0.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Point NOTICE to the current license filename

The new NOTICE text says See LICENSE for details, but this same commit renames LICENSE to LICENSE.txt, so the reference is now stale and can mislead downstream users or tooling that follows the notice for legal documentation. Update the notice to reference LICENSE.txt so attribution and license pointers stay internally consistent.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an Apache License 2.0 LICENSE.txt file preserving the upstream Opticks copyright notice, and a NOTICE.txt providing attribution to the Opticks project and pointers to historical authorship in Git.

Changes:

  • Add LICENSE.txt with the full Apache License 2.0 text and Opticks Authors copyright (2019).
  • Add NOTICE.txt attributing Simphony's lineage to the Opticks project and referencing the license.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
NOTICE.txt New attribution notice referencing Opticks and the project license.
LICENSE.txt Apache License 2.0 text with preserved upstream Opticks copyright header.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread NOTICE.txt Outdated
@plexoos plexoos self-assigned this May 14, 2026
@plexoos plexoos requested a review from ggalgoczi May 14, 2026 16:13
@plexoos plexoos added the documentation Improvements or additions to documentation label May 14, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants