ColossalAI icon indicating copy to clipboard operation
ColossalAI copied to clipboard

[BUG]: Missing copyright notices and licenses

Open groganz opened this issue 3 years ago • 1 comments

🐛 Describe the bug

Following best practices, all projects under https://github.com/hpcaitech should have a LICENSE file and all source code should include a copyright notice.

These projects lack a LICENSE file:

https://github.com/hpcaitech/CachedEmbedding https://github.com/hpcaitech/OPT-Benchmark https://github.com/hpcaitech/ColossalAI-Pytorch-lightning https://github.com/hpcaitech/luchen-website https://github.com/hpcaitech/TensorNVMe

The copyright notice should be added in the LICENSE file of each project, e.g. at https://github.com/hpcaitech/ColossalAI/blob/main/LICENSE#L190, presumably:

Copyright 2021- HPC-AI Technology Inc.

It should also be added on top of each source code file by including the appropriate source header also including the respective license information, e.g.

# Copyright 2021- HPC-AI Technology Inc.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#     http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

See https://www.apache.org/legal/src-headers.html

Environment

No response

groganz avatar Nov 16 '22 18:11 groganz

Hi @groganz Thanks for your suggestion. Would you please submit PRs to fix them? We very much welcome contributions from the open source community, thanks.

binmakeswell avatar Nov 17 '22 09:11 binmakeswell

Updated.

binmakeswell avatar Nov 22 '22 08:11 binmakeswell