Screen identifiers should not include the word 'screen'

[matthew-blackman]

Discussing this with @catherinecarter, we agreed that makes the identifiers too long. Let's use 'variability', 'sources', 'measures', 'sampling' instead of 'variabilityScreen', 'sourcesScreen', 'measuresScreen', 'samplingScreen'.

[matthew-blackman]

This has been implemented in the commit. Marking to cherry-pick.

[matthew-blackman]

The above commit uses the screen identifier values instead of tandem names, as well as adds support for the the screen identifier variable to the CODAP wrapper. Marking ready to cherry-pick.

[matthew-blackman]

Please close after verifying.

[KatieWoe]

Unless I'm misunderstanding this issue, this doesn't seem to have occurred

[catherinecarter]

@KatieWoe - I believe this issue is referring to what string will be passed into CODAP as the title of a column heading. I don't believe this is meant to refer to the PhET-iO ID in studio.

[KatieWoe]

In that case, what should I check in rc.2?

[catherinecarter]

Oh shoot! I meant the contents of the column not the column heading.

So what to look for... I believe it would be to make sure that if you launch projectiles on one screen, let's just say launcher 4 on the Variability Screen, and then you launch the same launcher from a different screen, say launcher 4 on the Measures Screen, then the contents of the "screen" column in the table would say, "Variability" and "Measures," respectively.

Does that make sense?

The purpose is to delineate which screen the launches came from, in case anyone wanted to recreate the histogram per screen without combining data in an unwanted way.

[KatieWoe]

We haven't been given anything to do with codap in these tests. I was told that would be next release. I don't know how to check this table.

[catherinecarter]

Gotcha. We will await instructions from @matthew-blackman and @samreid. I don't have the link to take a look myself. Thanks, Katie!

[KatieWoe]

Discussed on zoom. This was checked by examining the state using code from the examples doc. Things look ok, closing.