phetsims / energy-skate-park-basics

"Energy Skate Park: Basics" is an educational simulation in HTML5, by PhET Interactive Simulations.
http://phet.colorado.edu/en/simulation/energy-skate-park-basics
GNU General Public License v3.0
2 stars 9 forks source link

Discarding thermal energy leaves dot behind #449

Closed rea-laura closed 4 years ago

rea-laura commented 5 years ago

Test Device

Jordan

Operating System

Mojave 10.14.2

Browser

Chrome 71.0.3578.98

Problem Description

For https://github.com/phetsims/QA/issues/226: When all of the energy is consumed into thermal energy, discarding the thermal energy leaves behind a dot.

Steps to Reproduce

Method 1 - leaves dot with no energy left in bar graph 1) Go to energy skate park basics from phettest.colorado.edu 2) Select "Friction" page from the home menu 3) Set friction to "lots" 4) turn on pie chart and bar graph 5) drop skater on track and let them oscillate until all energy is converted into thermal. Wait a few seconds for skater to completely stop. 6) press the trash can by the thermal energy in the graph key to discard thermal energy 7) a dot will be left behind floating over the head of the skater, but no energy is left in the bar graph (image 1). Dot will also move when you adjust the size of the skater.

Method 2 - dot left behind with energy left in graph 1) using the same settings, place the skater in the bottom of the curve so it won't move. 2) grab the dot under the skater and move it slightly to one side. Release so it rolls down just a little bit 3) while the skater is still moving slightly, discard the thermal energy using the trash can button. Press trash can button repeatedly until it becomes inactivated. 4) energy will still be left behind in the graph (small amount of thermal and total) and a dot will be left above skater's head (image 2).

Method 2 - control. Leaves no dot above skater's head 1) with all the same settings on, drop the skater onto level ground 2) discard thermal energy 3) no dot above the skater's head (image 3)

Visuals

Image 1: dot above head, no energy in graph screen shot 2018-12-18 at 10 21 10 am

Image 2: dot above head, energy left in graph screen shot 2018-12-18 at 10 33 40 am

Image 3: no dot screen shot 2018-12-18 at 10 33 53 am

samreid commented 5 years ago

Thanks @rea-laura, I'm also assigning @jessegreenberg so he's aware of it and in case he addresses it before I get here.

UPDATE: @jessegreenberg can you please take the lead on this? Please reassign me or reach out to me if you would like collaboration or assistance.

jessegreenberg commented 4 years ago

Sounds good - I took a look and here is what I found:

jessegreenberg commented 4 years ago

I found https://github.com/phetsims/energy-skate-park-basics/issues/136 where the following was introduced to make the pie chart invisible when energies are too low.

        // If any value is too low, then don't show it, see #136
        const THRESHOLD = 1E-4;

        // if only one component of pie chart, then show as a circle so there are no seams
        const numberComponents = ( skater.potentialEnergyProperty.value > THRESHOLD ? 1 : 0 ) +
                               ( skater.kineticEnergyProperty.value > THRESHOLD ? 1 : 0 ) +
                               ( skater.thermalEnergyProperty.value > THRESHOLD ? 1 : 0 );

So I would recommend that we don't do anything further for this issue. The original issue report (no energy in bar graph, energy showed in pie chart) was definitely an issue. But that is no longer what is happening and issue history and https://github.com/phetsims/energy-skate-park-basics/issues/449#issuecomment-572297248 demonstrate that sim is currently working as expected.

Closing.