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

Surface temperature layer doesn't exactly match artwork below #251

Closed
Tracked by #885
Nancy-Salpepi opened this issue Jan 6, 2023 · 5 comments
Closed
Tracked by #885
Labels

Comments

@Nancy-Salpepi
Copy link

Test device
MacBook Air (m1 chip)

Operating System
macoS 13.0.1

Browser
safari 16.1

Problem description
For phetsims/qa#871, the surface temperature layer doesn't exactly match the scene below--leaving lighter areas in a few places where the surface and sky meet.

Visuals
Screenshot 2023-01-06 at 1 22 44 PM

Screenshot 2023-01-06 at 1 22 07 PM

Troubleshooting information: !!!!! DO NOT EDIT !!!!! Name: ‪Greenhouse Effect‬ URL: https://phet-dev.colorado.edu/html/greenhouse-effect/1.1.0-rc.1/phet/greenhouse-effect_all_phet.html? Version: 1.1.0-rc.1 2022-12-23 00:15:29 UTC Features missing: applicationcache, applicationcache, touch Flags: pixelRatioScaling User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.1 Safari/605.1.15 Language: en-US Window: 1399x710 Pixel Ratio: 2/1 WebGL: WebGL 1.0 GLSL: WebGL GLSL ES 1.0 (1.0) Vendor: WebKit (WebKit WebGL) Vertex: attribs: 16 varying: 30 uniform: 1024 Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32) Max viewport: 16384x16384 OES_texture_float: true Dependencies JSON: {}
@Nancy-Salpepi Nancy-Salpepi added the type:bug Something isn't working label Jan 6, 2023
@jbphet
Copy link
Contributor

jbphet commented Jan 6, 2023

I'm impressed that you noticed this! @samreid, @arouinfar, and I just reviewed this, and we feel that it doesn't compromise the pedagogy at all, so if it can be fixed quickly it is worth it, otherwise it should be left as is.

jbphet added a commit that referenced this issue Jan 10, 2023
@jbphet
Copy link
Contributor

jbphet commented Jan 10, 2023

I've updated this such that the green from the background never peeks through. This was, I think, the most noticeable issue. However, this has caused the edge of the orange glow to be a little above the horizon in a couple of places. These are generally less noticeable because they are often behind buildings and such.

@arouinfar - I've done as much as I can reasonably do on this. From here we can either decide that it is acceptable or we could have @mariahmaephet modify the artwork. I'll leave that up to you.

@arouinfar
Copy link
Contributor

However, this has caused the edge of the orange glow to be a little above the horizon in a couple of places. These are generally less noticeable because they are often behind buildings and such.

I agree. I reviewed with @jbphet and we decided that this version is preferable. In order to match things perfectly, we'd need to work with the graphic artist to update the assets. However, we don't think this cost would be worth the benefit.

@Nancy-Salpepi
Copy link
Author

It looks much better in rc.2
There are still a couple of spots where I can see the lighter color in the Ice Age scene, not sure if they are worth the time.

Screenshot 2023-01-20 at 3 19 01 PM

Screenshot 2023-01-20 at 3 26 32 PM

Feel free to close if you don't think it is worth fixing.

@arouinfar
Copy link
Contributor

Feel free to close if you don't think it is worth fixing.

I don't think it's worth fixing because getting a 100% perfect match requires updating the artwork. I do, however, appreciate your attention to detail @Nancy-Salpepi!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants