-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
prepare transition #155
base: main
Are you sure you want to change the base?
prepare transition #155
Conversation
@@ -1,6 +1,7 @@ | |||
BSD 3-Clause License | |||
|
|||
Copyright (c) 2020, Philip Meier | |||
Copyright (c) 2020-2024, Philip Meier | |||
Copyright (c) 2024-, Slicer Community |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going by the suggestion from #152 (comment). SlicerPyTorch uses
This Agreement covers contributions to and downloads from the 3D
Slicer project ("Slicer") maintained by The Brigham and Women's
Hospital, Inc. ("Brigham")
LMK what you prefer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcfr, thoughts on "Slicer Community" being listed here? I don't think we necessarily need to bring Brigham into the conversation here as this is not directly Slicer related or adopting the Slicer license.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In practice, copyright needs to be associated with a legal entity. So here are some options:
Copyright (c) 2024-, Brigham and Women's Hospital (BWH)
or
Copyright (c) 2024-, Kitware, Inc.
Copyright (c) 2024-, Revvity, Inc.
That said, we could also consider specifying copyright as significant contribution are made ?
maintainers = [ | ||
{ name = "Slicer Community"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LMK if you want to add a contact email here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be @jamesobutler and myself (@jcfr)
Thanks for this draft @pmeier. We can integrate the changes as-is and I make updates to the listed author upon further discussions with the other Slicer devs prior to cutting a new release of this package. Or we can wait for those discussions to complete. To not bother you with this transfer, if you can provide me Admin role of this repository I will transfer it into the Slicer GitHub organization. All links to https://github.com/pmeier/light-the-torch will automatically redirect to the future https://github.com/Slicer/light-the-torch regardless if this PR is integrated before the transfer or not. |
Closes #152. Cc @jamesobutler