-
Notifications
You must be signed in to change notification settings - Fork 144
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
bexvar and eROSITA data #582
Comments
@JohannesBuchner thank you very much for sharing your new method and code! These would indeed be great additions. |
@JohannesBuchner thanks again. We are proposing this as a Google Summer of Code project: |
@JohannesBuchner: @mihirtripathi97 was accepted for GSoC and is now in charge of implementing Bexvar in Stingray. |
Sorry, somehow I missed your ping. I was under the impression that licences like GPL apply per project, while licenses like BSD and MIT have to be copied into each file as a header. Even for GPL a short header is usually done (see for example Mozilla source code, or GPL instructions). But apparently this is not necessary (or people disagree about it): https://softwareengineering.stackexchange.com/questions/125836/do-you-have-to-include-a-license-notice-with-every-source-file https://stackoverflow.com/questions/845895/putting-license-in-each-code-file Is there a way to avoid this code being namelessly swallowed? It would be nice to have the authorship in that file, i.e, the authors of bexvar plus whoever worked on it from your side. In the case of scipy, they made v1.0 paper at some point, and invited everyone with substantial code contributions to be on the co-author list. I don't know if you have any plans in that direction in the future. If the points above sound reasonable to you, then I'd be happy for this code to be merged under MIT licence.
|
Hi @JohannesBuchner, inviting you in a future paper and as co-author of the next release on Zenodo and the other platform is the least we can do! I don't know how to preserve partial code contributions on git though, we did cite the relevant papers and the original code in the documentation. |
Oh, I was thinking more of adding a copyright header to the file. Didn't think of the git commit (and don't care too much about it). |
How do we merge the original contributions with the new ones? Something like this?
|
Sounds fine |
Done: 9f48fbb |
Thank you! |
I guess the remainder of this issue is on ingestion of eROSITA data. |
@JohannesBuchner do you have example eROSITA data you would like to analyze? |
There are some example light curves in https://github.com/JohannesBuchner/bexvar/tree/main/examples |
Dear all,
I wanted to point out to you:
So feel free to
Cheers,
Johannes
The text was updated successfully, but these errors were encountered: