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

credits #2

Closed
Tracked by #573
pixelzoom opened this issue Jul 11, 2019 · 12 comments
Closed
Tracked by #573

credits #2

pixelzoom opened this issue Jul 11, 2019 · 12 comments

Comments

@pixelzoom
Copy link
Contributor

pixelzoom commented Jul 11, 2019

Update credits in natural-selection-main.js.

pixelzoom added a commit that referenced this issue Oct 23, 2019
@pixelzoom
Copy link
Contributor Author

Credits in the design doc are:

Amanda McGarry (lead designer)
Chris Malley (developer)
Megan Hoffman
Oliver Nix
Ariel Paul
Kathy Perkins

Credits in the Java version are:

screenshot_1634

PhET convention is to move the Java credits to "Team". So the HTML5 credits are initially:

screenshot_1633

@amanda-phet
Copy link
Contributor

PhET convention is to move the Java credits to "Team".

Is this new? I am used to seeing original developers and designers listed in those respective categories, not put in the team. Looking at the recently published PhET sim ports, Gas Props follows this convention, but Blackbody, EFAC, Fractions Intro, Build a Fraction, do not. Curve Fitting similarly doesn't follow this convention.

@ariel-phet can you clarify.

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Nov 4, 2019

I don't know what other developers have been doing. But if I reuse or port parts of the Java sim, then I definitely list the original Java developer as "developer" because their code is part of the HTML5 implementation. That was certainly the case in Gas Properties, specifically for collision detection. That is not likely to be the case for Natural Selection. And I'll be happy to revisit when HTML5 implementation is done.

@arouinfar
Copy link

@amanda-phet the Java version of Gas Properties does not explicitly credit anyone as the lead designer, which is why I'm the only lead listed in the HTLM5 credits. Generally speaking, I'm always co-listed with the Java lead in HTML5 credits.

@pixelzoom sounds like you intended to credit Ron LeMaster as a developer in Gas Props, but he is in the Team section of the credits.

@pixelzoom
Copy link
Contributor Author

Hmm... I guess I did strictly follow the "move Java credits to team" policy. I don't think Ron cares, I'll buy him a beer ;)

If someone wants to specify a different policy, please do and I'll follow it. And if the checklist is being followed, there are multiple opportunities to adjust the credits before a sim is published. That's why this issue exists.

@ariel-phet
Copy link

ariel-phet commented Nov 11, 2019

@pixelzoom @amanda-phet @arouinfar my understanding went something like this...

For ports, we list the original lead designer in the Lead Design, but generally if there have been significant changes, we put the HTML5 lead first. (So for instance, current Vector Addition lists Mike Dubson first in lead design, but probably Amy should be listed first there)

For software development, if the code is basically the same as the Legacy code, it seems correct to include the original software developer, but certainly recently, most of the code has basically been written from scratch. I think this is mostly developer discretion.

In team, we just list people alphabetically.

I cannot say I have personally been too worried as long as people are listed (but that is because I personally don't care about credits too much). But the above is my understanding of our decisions.

@pixelzoom
Copy link
Contributor Author

pixelzoom commented Nov 11, 2019

Thanks @ariel-phet.

@amanda-phet @arouinfar let me know if you want to make changes to the "Lead Design" credit in Natural Selection and Vector Addition respectively. (EDIT: No changes required to Vector Addition credits, see phetsims/vector-addition#98 (comment).)

I'll decide on the "Software Development" credit when we see whether any of the Java implementation gets incorporated into the HTML5 version (so far not).

@pixelzoom pixelzoom removed their assignment Nov 11, 2019
pixelzoom added a commit that referenced this issue Aug 4, 2020
pixelzoom added a commit that referenced this issue Aug 4, 2020
@pixelzoom
Copy link
Contributor Author

pixelzoom commented Aug 4, 2020

I've added @phet-steele to the credits, for his role in testing the prototype release.

This was referenced Aug 20, 2020
pixelzoom added a commit that referenced this issue Sep 27, 2020
@pixelzoom
Copy link
Contributor Author

@KatieWoe I've added Logan Bray to the QA credits, for his role in 1.2 RC testing. (Unfortunately he will not appear in the 1.2 credits, but it's not a public release, it's just a link that's going to a client for temporary use.). Anyone else that I should add?

pixelzoom added a commit that referenced this issue Sep 27, 2020
@KatieWoe
Copy link
Contributor

I think Liam should be added as well.

pixelzoom added a commit that referenced this issue Sep 28, 2020
@pixelzoom
Copy link
Contributor Author

I've added Liam.

@pixelzoom
Copy link
Contributor Author

Looks like I forgot to close this after publishing 1.2. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants