search

antego / radon

Implementation of Radon algorithm in C++ with Qt interface
MIT License
5 stars 5 forks source link

Code lacks LICENSE file #3

Open cdalitz opened 5 years ago

cdalitz commented 5 years ago

Many thanks for uploading an implementation of the radon transform on github.

Unfortunately, the code does not contain information about the licensing terms for reuse. Can you please add a file LICENSE (e.g. a two clause BSD license, or what you prefer) to clarify whether modification and use of the code in an FOSS licensed project is permitted?

antego commented 5 years ago

Done. Any other suggestions on how to make this code more accessible for community?

cdalitz commented 5 years ago

Thanks for the quick response! Concerning your question about how to make it more accessible: maybe a brief comment in the Readme.md would be useful how only the radon transform can be used (without the need to compile the Qt GUI). I have noticed that radon.cpp includes a Qt header, although, I think, this is not necessary because the implementation only relies on OpenCV, but not on Qt.

antego commented 5 years ago

Thanks for the feedback. Will improve it soon.