Skip to content

License fix#77

Merged
Dinghye merged 1 commit into
cybergis:mainfrom
adamjstewart:license
Apr 24, 2026
Merged

License fix#77
Dinghye merged 1 commit into
cybergis:mainfrom
adamjstewart:license

Conversation

@adamjstewart
Copy link
Copy Markdown
Contributor

Summary

rs-embed contains a lot of vendored libraries under both Apache-2.0 and MIT licenses. Thus, the license of the entire library should be "Apache-2.0 AND MIT".

Related Issue

N/A

Testing

  • I ran the relevant tests locally.
  • I updated docs for user-facing behavior changes.
  • I updated CHANGELOG.md for user-facing API, model, semantic, or installation changes.
  • This PR does not need a changelog entry.

Notes

You probably also need to add a copyright header to all vendored files. I don't think I saw any files copied from TorchGeo (technically you don't even need LICENSE.torchgeo) but this would look like:

# Copyright (c) TorchGeo Contributors. All rights reserved.
# Licensed under the MIT License.

if there were any files from TorchGeo.

Signed-off-by: Adam J. Stewart <ajstewart426@gmail.com>
@Dinghye
Copy link
Copy Markdown
Collaborator

Dinghye commented Apr 24, 2026

Thank you for your reminder, Adam! I will check them now.

@Dinghye Dinghye merged commit cddf802 into cybergis:main Apr 24, 2026
5 of 6 checks passed
@adamjstewart adamjstewart deleted the license branch April 24, 2026 21:34
@adamjstewart
Copy link
Copy Markdown
Contributor Author

Also let me know if we can add any of these vendored models to TorchGeo to help. We can easily move any MIT-licensed code to TorchGeo so you don't have to maintain it. Then things could be 100% Apache-2.0.

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.

2 participants