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

added hack week paper to readme #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

dhuppenkothen
Copy link
Owner

@dhuppenkothen dhuppenkothen commented Apr 4, 2019

Addresses #57.

@bsipocz is this what you had in mind?


This change is Reviewable

Copy link

@bsipocz bsipocz left a comment

Choose a reason for hiding this comment

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

Yes, this is exactly I had in mind. Thanks!

README.md Outdated
@@ -20,6 +20,8 @@ The targets could be based on various objectives that depend on the workshop or

Note that this is explicitly not the same as a quota: the underlying algorithim optimizes for the subset of participants that overall matches up with the targets, taking into account all targets simultaneously. It is also possible to include relative weights between targets, depending on how important they are to you.

For some more information about participant-driven events and some considerations important in the context of participant selection for these events can be found in [this paper](https://www.pnas.org/content/115/36/8872) (see in particular the appendix for practical advice on some aspects).
Copy link

Choose a reason for hiding this comment

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

I was told that instead of "this paper"/"here"/etc it may be a little bit more useful to say something about the paper ("in our paper published in PNAS", or similar, I'm not good with words).

Copy link
Owner Author

Choose a reason for hiding this comment

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

Check out the new version?

Copy link

Choose a reason for hiding this comment

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

much better, thanks!

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.

2 participants