Skip to content

AWG Software Policy#12

Merged
mhasself merged 8 commits into
masterfrom
awg_guidance
Oct 13, 2022
Merged

AWG Software Policy#12
mhasself merged 8 commits into
masterfrom
awg_guidance

Conversation

@msyriac

@msyriac msyriac commented Apr 16, 2021

Copy link
Copy Markdown
Member

...described in Chapter 10 "Special Guidance for AWGs".

@msyriac msyriac requested review from aiolasimone and mhasself April 16, 2021 14:54
Comment thread source/awg.rst Outdated
Codes on SO Github
==================

If a code repository is hosted on the SO Github and is publicly visible, then it may be used by anyone (e.g. the public, ACT, Polarbear) without requiring explicit permission, unless the license somehow forbids this.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might be aware and on purpose. But, I just wanted to point out that, if I correctly remember, the opposite is legally true, that is, other people cannot use the code in a public repository unless the license permits.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I wasn't aware. BSD, MIT and GPL licenses do permit this right? If yes, I will clarify this. Thanks!

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, BSD and MIT are the most permissive licenses; they are often used for academic code. GPL is more restrictive.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok fixed.

@aiolasimone aiolasimone left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I brought up just a couple of minor points,

Comment thread source/awg.rst Outdated
Comment thread source/awg.rst Outdated
Codes on SO Github
==================

If a code repository is hosted on the SO Github and is publicly visible, then it may be used by anyone (e.g. the public, ACT, Polarbear) without requiring explicit permission, if the license allows this. The licenses that we recommend (`BSD 2-Clause License`_ or `MIT License`_) do allow this.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This point is fair. I am just wondering if we are entering "Publication committee" land.
For example in ACT, there are periods following a public data release during which you cannot publish your own paper based on public ACT data unless you give the chance to other members to opt-in. If SO does something similar (and I am not sure), maybe they want to extend that to codes too?. In other words, I would put a warning for now saying that we would like to discuss with PP? In general I am in favor of your proposal though

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since there has been some new activity ...

Not sure if this item ended up getting discussed with PP or not, but unless they deferred entirely to us then I think we should mention that they get final say on what sort of analyses of what data can be published by whom at what times.

@mhasself mhasself left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good.

Cosmetic request -- can you "fill" the paragraphs? Rationale:

  • The raw text document also needs to look nice.
  • It is easier to process diffs, later, if the lines are short.

@mhasself mhasself merged commit 6166ec7 into master Oct 13, 2022
@mhasself mhasself deleted the awg_guidance branch October 13, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants