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

Instrumentation needs for Sonification/GT collaboration #140

Closed
ariel-phet opened this issue Nov 22, 2016 · 11 comments
Closed

Instrumentation needs for Sonification/GT collaboration #140

ariel-phet opened this issue Nov 22, 2016 · 11 comments
Assignees

Comments

@ariel-phet
Copy link

ariel-phet commented Nov 22, 2016

Related to #107

@jbphet said:

Sam, Emily, Jesse and I met with Brianna T from Georgia Tech today to discuss her needs for the BAA instrumentation. Here are a couple of things that you may want to know from this meeting.

  • She figures they will need a version that supplies basic parameters for sonification in the Feb 2017 time frame. The parameters she requested are, for the most part, accessible in the simulation and shouldn’t be hard to instrument. In fact, many of them may already be instrumented.

  • The project where they control the simulation via alternate user interface devices (Leap?) will be later in the year. She didn’t have a clear sense of when, but I got the sense that it could be several months after that.

  • They are not currently planning to sonify the game.

@jbphet
Copy link
Contributor

jbphet commented Dec 6, 2016

Brianna T also said:

The variables we've been talking about for sonifications in Build an Atom are:

  1. number of protons/neutrons/electrons
  2. net charge (+/- ion or neutral)
  3. stable/unstable
  4. mass number

Knowing the element name is good, but we don't necessarily need it for sonifications. I think we could technically calculate something like #2-4, but if you've got a variable tracking it already we can just use that.

@jbphet
Copy link
Contributor

jbphet commented Dec 6, 2016

This commit added ability to see 3 or the 4 values listed in the comment immediately above: a8e4046.

@jbphet
Copy link
Contributor

jbphet commented Dec 6, 2016

Issue #107 is also about adding instrumentation, and is a more general version of this issue.

jbphet added a commit that referenced this issue Dec 6, 2016
jbphet added a commit that referenced this issue Dec 6, 2016
…h phet-io, see #107 and #140, also did some other cleanup
@jbphet
Copy link
Contributor

jbphet commented Dec 6, 2016

I've published a version that I believe has the instrumentation necessary to support the items requested in comment #140 (comment). That version can be found here: http://www.colorado.edu/physics/phet/dev/html/build-an-atom/1.3.0-phetiodev.1/

Assigning to @ariel-phet to determine whether we should test this before handing it over to Georgia Tech. It seems to me like some amount of testing would probably be good.

@ariel-phet
Copy link
Author

Yes, some testing would be good, please make a task, provide some context for QA and assign to me initially.

@jbphet
Copy link
Contributor

jbphet commented Dec 8, 2016

Task created, see phetsims/tasks#748.

@jbphet
Copy link
Contributor

jbphet commented Jan 11, 2017

The testing has been completed. After talking with @samreid, I learned that I really should have done an RC and production version for sharing with GT (this is apparently our policy for sharing phet-io versions), so I went ahead and created these versions and had them sanity tested. The link has been provided to @emily-phet, and I will assign this to her so that she can either log any problems that arise once the GT folks start using it or close the issue if it meets their needs.

@jbphet
Copy link
Contributor

jbphet commented Jan 11, 2017

By the way, I'm not going to put the link to the phet-io version in this GitHub issue since the issue is visible to the general public and the link is intended to be hidden.

@jbphet jbphet assigned emily-phet and unassigned jbphet Jan 11, 2017
@jbphet
Copy link
Contributor

jbphet commented Mar 8, 2017

In a meeting with Brianna and others today, she expressed a need to be able to monitor the open/closed state of the accordion boxes. This capability has been implemented in the 1.4.0-phetio release, but that release is not yet fully tested and is therefore not yet a production release. I've let Brianna know about this and will provide her the production release when it's ready.

@emily-phet
Copy link

@jbphet can this be closed?

@emily-phet emily-phet assigned jbphet and unassigned emily-phet Jan 7, 2018
@jbphet
Copy link
Contributor

jbphet commented Jan 8, 2018

Sure. Closing.

@jbphet jbphet closed this as completed Jan 8, 2018
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

3 participants