Skip to content
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

Open data in science #410

Merged
merged 1 commit into from
Dec 15, 2016
Merged

Conversation

gordondavies
Copy link
Contributor

See the changes

@Floppy
Copy link
Member

Floppy commented Aug 7, 2016

👍 on the principle, though I wonder if the wording needs to be slightly more precise to focus on publicly-funded research? Should commercial research need to be published in the same way? Trials and negative results should, of course, but I'm not sure of the nuances.

@gordondavies
Copy link
Contributor Author

Definitely needs more careful consideration and wording, and specific clarification on public and commercial research requirements. I'll make some changes.

@Floppy
Copy link
Member

Floppy commented Aug 9, 2016

Great - the custom editor doesn't support updating existing PRs, but given your github history I assume you know what you're doing :)

@philipjohn
Copy link
Member

👍 commercial research could probably describe the pharmaceutical industry so there is something in applying the same rules to medical research, certainly. Probably a wording thing.

@Floppy
Copy link
Member

Floppy commented Oct 26, 2016

@ghaggart are you still intending to make some changes to this proposal?

@gordondavies
Copy link
Contributor Author

@Floppy Apologies for the late reply on this, I'll have a look at the wording and make the changes asap.

@philipjohn
Copy link
Member

This proposal failed because of time :(

I wonder if changes will reset the clock? @Floppy

@Floppy
Copy link
Member

Floppy commented Dec 15, 2016

It won't, but we could merge anyway as technically the thing passed, we just knew @ghaggart was intending to make some changes. We could do those as a separate PR instead. Our future automated system would have merged it. So we should. merges

@Floppy Floppy merged commit cdde0c4 into openpolitics:master Dec 15, 2016
@Floppy
Copy link
Member

Floppy commented Dec 15, 2016

@ghaggart if you still want to make changes to this, then do so in another edit. :)

@Floppy
Copy link
Member

Floppy commented Feb 8, 2017

This proposal is open for discussion and voting. If you are a contributor to this repository (and not the proposer), you may vote on whether or not it is accepted.

How to vote

Vote by entering one of the following symbols in a comment on this pull request. Only your last vote will be counted, and you may change your vote at any time until the change is accepted or closed.

vote symbol type this points
Agree 👍 :thumbsup: 1
Abstain :hand: -1
Block 👎 :thumbsdown: -1000

Proposals will be accepted and merged once they have a total of 2 points when all votes are counted. Votes will be open for a minimum of 7 days, but will be closed if the proposal is not accepted after 90.

Votes are counted automatically here, and results are set in the merge status checks below.

Changes

If the proposer makes a change to the proposal, no votes cast before that change will be counted.

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