Add basic README for auxiliary repository#44
Conversation
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: neuronflow <7048826+neuronflow@users.noreply.github.com>
neuronflow
left a comment
There was a problem hiding this comment.
@copilot remove this section ## Contributing
We welcome all kinds of contributions from the community!
Reporting Bugs, Feature Requests and Questions
Please open a new issue here.
Code Contributions
Nice to have you on board! Please have a look at our CONTRIBUTING.md file.
Co-authored-by: neuronflow <7048826+neuronflow@users.noreply.github.com>
neuronflow
left a comment
There was a problem hiding this comment.
@copilot also show how to read and write DICOM files
maybe an i/o section for each format, nifti, DICOM, tiff
Co-authored-by: neuronflow <7048826+neuronflow@users.noreply.github.com>
Reorganized the Usage section with separate I/O sections for NIfTI, DICOM, and TIFF formats. Added examples showing how to read DICOM series and write to DICOM format (with optional reference DICOM for metadata) in be6e583. |
neuronflow
left a comment
There was a problem hiding this comment.
@copilot also show how to write numpy arrays to DICOM with and without reference images
…ence images Co-authored-by: neuronflow <7048826+neuronflow@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [](https://pypi.org/project/auxiliary/) | ||
| [](https://pypi.org/project/auxiliary/) | ||
| [](http://auxiliary.readthedocs.io/?badge=latest) | ||
| [](https://opensource.org/licenses/Apache-2.0) |
There was a problem hiding this comment.
The License badge indicates "Apache 2.0" but according to pyproject.toml (line 13), the project is licensed under "AGPL-3.0". The badge should be updated to reflect the correct license.
| [](https://opensource.org/licenses/Apache-2.0) | |
| [](https://www.gnu.org/licenses/agpl-3.0.html) |
There was a problem hiding this comment.
@copilot no in this case please change the project license to Apache 2.0
There was a problem hiding this comment.
Updated pyproject.toml license from AGPL-3.0 to Apache-2.0 in f170a74. The LICENSE file already contained the Apache 2.0 license, so this makes everything consistent.
Co-authored-by: neuronflow <7048826+neuronflow@users.noreply.github.com>
neuronflow
left a comment
There was a problem hiding this comment.
lgtm but I am quite sure things could be better :)
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
neuronflow
left a comment
There was a problem hiding this comment.
lgtm anything is better than no readme.
Added citation section with reference for the auxiliary framework.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Creates a basic README.md file for the auxiliary repository, inspired by the BrainLesion/PeTu repository structure.
Changes Made
Fixes #37
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.