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 (add right before RC.1) #171

Closed
Tracked by #123
samreid opened this issue Jun 5, 2024 · 10 comments
Closed
Tracked by #123

credits (add right before RC.1) #171

samreid opened this issue Jun 5, 2024 · 10 comments

Comments

@samreid
Copy link
Member

samreid commented Jun 5, 2024

No description provided.

@samreid samreid mentioned this issue Jun 5, 2024
84 tasks
@samreid samreid changed the title credits (will not be completed until after RC testing), see {{GITHUB_ISSUE_LINK}} credits (add right before RC.1) Jun 5, 2024
@samreid
Copy link
Member Author

samreid commented Aug 4, 2024

@DianaTavares can you please recommend the credits for Buoyancy and Buoyancy Basics? Density is currently written like so:

  credits: {
    leadDesign: 'Diana L\u00f3pez Tavares, Kelly Lancaster',
    softwareDevelopment: 'Jonathan Olson, Chris Malley (PixelZoom, Inc.), Sam Reid',
    team: 'Wendy Adams, Trish Loeblein, Ariel Paul, Archie Paulson, Kathy Perkins, Noah Podolefsky, Amy Rouinfar, Carl Wieman',
    qualityAssurance: 'Logan Bray, Steele Dalton, Jaron Droder, Brooklyn Lash, Emily Miller, Liam Mulhall, Devon Quispe, Jacob Romero, Nancy Salpepi, Katie Woessner',
    graphicArts: '',
    soundDesign: '',
    thanks: ''
  },

@DianaTavares
Copy link

credits: {
   leadDesign: 'Diana L\u00f3pez Tavares, Kelly Lancaster',
   softwareDevelopment: 'Michael Kauzmann, Jonathan Olson, Sam Reid, Agustin Vallejo'
   team: 'Wendy Adams, Trish Loeblein, Chris Malley (PixelZoom, Inc.), Ariel Paul, Archie Paulson, Kathy Perkins, Noah Podolefsky, Amy Rouinfar, Carl Wieman',
   qualityAssurance: 'Jaron Droder, Nancy Salpepi, Katie Woessner',
   graphicArts: '',
   soundDesign: '',
   thanks: ''
 },

Remember to add the Agustín, the accent!

@DianaTavares DianaTavares assigned samreid and unassigned DianaTavares Aug 5, 2024
@samreid
Copy link
Member Author

samreid commented Aug 7, 2024

Should the credits for density also be changed? If so, how? Should it use the same as in the preceding comment?

@samreid samreid assigned DianaTavares and unassigned samreid Aug 7, 2024
@DianaTavares
Copy link

These are the ones in Density:
image

And yes, I think that the best is to put the same as Buoyancy, in that way we add Michael and Agustín and delete from the QA the people that are not at PhET right now.

I guess that for changing the ones on the webpage is something to do in Admin. I will ask @arouinfar.

Process changing the credits in Density!

@DianaTavares DianaTavares assigned samreid and unassigned DianaTavares Aug 9, 2024
@samreid
Copy link
Member Author

samreid commented Aug 9, 2024

delete from the QA the people that are not at PhET right now.

It was my understanding that once someone appeared in the credits, they would remain there for all future publications of that sim. Just wanted to double check, it could be I misunderstood or that the plan changed. @arouinfar do you know one way or the other?

@samreid samreid assigned arouinfar and unassigned samreid Aug 9, 2024
@arouinfar
Copy link

It was my understanding that once someone appeared in the credits, they would remain there for all future publications of that sim. Just wanted to double check, it could be I misunderstood or that the plan changed. @arouinfar do you know one way or the other?

That's also my understanding @samreid. We should not remove any previous QA credits for Density.

Let's consider factoring out the credits to be shared across all of the density-buoyancy-common sims, as we've done in other sim suites. The legacy Density and Buoyancy sim pages both list the same people in the credits, and in HTML5, the design and development credits are the same for all 3 sims. It seems easiest to maintain one set of credits, especially when it comes time to start adding in QA credits as we near publication. Do we want to keep track of which subset of sims the testers worked on? This also means that Density's original QA team will be credited in Buoyancy/B:B, but that seems reasonable, as many of the bugs patched in Density likely had downstream effects in Buoyancy/B:B.

@arouinfar arouinfar assigned samreid and unassigned arouinfar Aug 12, 2024
samreid added a commit to phetsims/density that referenced this issue Aug 13, 2024
samreid added a commit to phetsims/buoyancy that referenced this issue Aug 13, 2024
samreid added a commit to phetsims/buoyancy-basics that referenced this issue Aug 13, 2024
samreid added a commit that referenced this issue Aug 13, 2024
@samreid
Copy link
Member Author

samreid commented Aug 13, 2024

I updated the credits based on the discussion above. I left Chris M in the software development since he was listed there in density. I sorted developers by last name. The result is:

{
  leadDesign: 'Diana L\u00f3pez Tavares, Kelly Lancaster',
  softwareDevelopment: 'Michael Kauzmann, Chris Malley (PixelZoom, Inc.), Jonathan Olson, Sam Reid, Agust\u00edn Vallejo',
  team: 'Wendy Adams, Trish Loeblein, Ariel Paul, Archie Paulson, Kathy Perkins, Noah Podolefsky, Amy Rouinfar, Carl Wieman',
  qualityAssurance: 'Logan Bray, Steele Dalton, Jaron Droder, Brooklyn Lash, Emily Miller, Liam Mulhall, Devon Quispe, Jacob Romero, Nancy Salpepi, Katie Woessner',
  graphicArts: '',
  soundDesign: '',
  thanks: ''
}

This is factored out to one place. @DianaTavares or @arouinfar any other changes to make?

@DianaTavares
Copy link

Looks good for me!

@DianaTavares DianaTavares removed their assignment Aug 13, 2024
@arouinfar
Copy link

Thanks @samreid, looks good. We'll need to finalize QA credits in RC, so I'll leave this issue open and unassigned.

@zepumph
Copy link
Member

zepumph commented Aug 22, 2024

Let's do QA Credits in #353

@zepumph zepumph closed this as completed Aug 22, 2024
@zepumph zepumph mentioned this issue Aug 22, 2024
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

4 participants