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

Concentration readout for time periods should be accurate #219

Closed
arouinfar opened this issue Oct 19, 2022 · 11 comments
Closed

Concentration readout for time periods should be accurate #219

arouinfar opened this issue Oct 19, 2022 · 11 comments

Comments

@arouinfar
Copy link
Contributor

arouinfar commented Oct 19, 2022

Looks like 2020 data appears, but 1950, 1750, and Ice Age data needs to be added. This should be done before publishing a prototype in #220. Self-assigning to confirm data.

image

@arouinfar arouinfar self-assigned this Oct 19, 2022
@arouinfar
Copy link
Contributor Author

@jbphet I added the data for the greenhouse gas concentrations to the design doc. Over to you to update the readouts for each time period.

@arouinfar arouinfar assigned jbphet and unassigned arouinfar Oct 25, 2022
@jbphet
Copy link
Contributor

jbphet commented Nov 16, 2022

I've added in the data that @arouinfar supplied, and we reviewed it in the context of the sim in this week's design meeting, and here are the follow ups:

  • The position of the data shouldn't shift when going to different dates (there is currently a big jump for the Ice Age date due to the narrow string for H2O). @jbphet will follow up on this.
  • Can we just get rid of the H2O indication? We have a number of questions and concerns about it. @jbphet will take it out for now, and @arouinfar will follow up with some research to make sure we're good with this

@jbphet
Copy link
Contributor

jbphet commented Nov 17, 2022

@arouinfar - The relative humidity portion has been removed. I'm assigning this to you for follow-up on whether we're good to go with that change.

@jbphet jbphet removed their assignment Nov 17, 2022
@jbphet
Copy link
Contributor

jbphet commented Nov 30, 2022

In a recent design meeting, it was brought up that using decimal values of ppm, as we do now, is unusual. It is more common to use ppb. Should we do that here? If we do, should we have a little info dialog like we do in other sims (e.g. Fourier).

Screenshot of current indicator:

image

@jbphet
Copy link
Contributor

jbphet commented Nov 30, 2022

@arouinfar said during today's design meeting to go ahead with ppb for the upcoming prototype release. We won't worry about having an info dialog for now and will follow up on whether it's needed for the full release later.

@jbphet jbphet self-assigned this Nov 30, 2022
jbphet added a commit that referenced this issue Dec 1, 2022
@jbphet
Copy link
Contributor

jbphet commented Dec 1, 2022

I've changed the designated compounds to use ppb. Unassigning myself, @arouinfar can follow up on whether we want to add an info dialog.

@arouinfar
Copy link
Contributor Author

Thanks @jbphet. The readout looks good in master.

Mark provided this source to explain water vapor's relationship to the greenhouse effect:
https://climate.nasa.gov/ask-nasa-climate/3143/steamy-relationships-how-atmospheric-water-vapor-amplifies-earths-greenhouse-effect/

We decided to remove the humidity readout for a few reasons:

  • Water vapor is the most abundant greenhouse gas in the atmosphere (likely the reason it was included in Java), but it is not the driver of climate change. As the global temperature rises, the atmosphere is able to hold more water vapor, so displaying the same relative humidity at all time periods is odd.
  • In Java, we listed the water vapor as 70% relative humidity. This number seems arbitrary. Humidity widely varies across locations and seasons. Global average humidity data isn't tracked in the same way as other greenhouse gases, for example, https://www.co2levels.org/

I've added a note to the Teacher Tips issue, so we can close this issue.

@Nancy-Salpepi
Copy link

@arouinfar I know this issue is closed, but I would like to vote for an info button in the full release. In my experience, HS students tend to look at numbers and not units (although they should). They just think biggest number = highest concentration (which is incorrect in this case).

@jbphet
Copy link
Contributor

jbphet commented Jan 31, 2023

Reopening based on what @Nancy-Salpepi said in the comment just above. @arouinfar - does her comment change your thinking on this, or do you still feel like the teacher tips update will cover it?

@pixelzoom
Copy link
Contributor

pixelzoom commented Jun 13, 2023

The open question is whether an Info dialog is needed here. @arouinfar?

@arouinfar
Copy link
Contributor Author

Thanks for the feedback @Nancy-Salpepi. We are using the standard units for these measurements, but I agree, students don't always read the units. That said, I think an info dialog explaining ppb vs. ppm is a bit heavy-handed and adds additional work for Interactive Description and PhET-iO. I would prefer to leave this as-is, and if we receive teacher feedback that students have issues with this, we can reconsider and include it in a future release.

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