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

Energy chunks returning to beaker can disappear #214

Closed
chrisklus opened this issue Feb 11, 2019 · 6 comments
Closed

Energy chunks returning to beaker can disappear #214

chrisklus opened this issue Feb 11, 2019 · 6 comments
Assignees
Labels

Comments

@chrisklus
Copy link
Contributor

Because of the clipping region in place on the beakers (see #114), energy chunks that are owned by the beakers can only be seen within the red column shown below. When a beaker is receiving an energy chunk from the air, it owns that chunk as it's on its way down, so if the beaker is moved away from the traveling chunk, then the chunk disappears until it reaches the beaker and magically appears in the fluid. This bug is present on master and 1.0.

kapture 2019-02-11 at 12 09 23

The reason this is not an issue for energy chunks that the beaker releases is because the air owns those chunks as they ascend, so they do not need to remain within the beaker's clipping region in order to be seen.

kapture 2019-02-11 at 12 14 44

Assigning to @jbphet and me to fix, and @ariel-phet for prioritization.

@ariel-phet
Copy link

I am going to put this one as high priority since the fundamental goal of the sim is energy conservation, so it seems a problematic bug from that standpoint.

@ariel-phet ariel-phet removed their assignment Feb 11, 2019
@jbphet
Copy link
Contributor

jbphet commented Feb 15, 2019

I've added code that causes the energy chunks to move with the destination object in the horizontal direction in this situation. @chrisklus and I both thought it looked pretty good. Marking as fixed, but will leave open and have QA verify in the next release.

@KatieWoe
Copy link
Contributor

This seems to be ok in 1.1.0-rc.2. I did notice that the last bit of the the energy chunk falling is much faster if moving the block/beaker at the time. I believe I also saw the energy chunk show this behavior by chance since it spawned near the clip area and zigzagged in and out. However, I haven't been able to replicate this behavior since it seems to happen by chance.

@KatieWoe
Copy link
Contributor

Update. Please disregard the second part of the above comment. I think I am misremembering and this actually happened on the the old version while I was testing to see what the bug looked like. I'll keep trying for a bit, but unless I can reproduce only speeding up chunks from the comment above should be considered. Sorry for the confusion @chrisklus

@KatieWoe
Copy link
Contributor

Haven't been able to reproduce on this rc

@chrisklus
Copy link
Contributor Author

Thanks @KatieWoe! The returning energy chunks speeding up when moving the beaker is intentional. That's great that you haven't seen them zigzag in and out of the clipping area.

I think we're ready to close here.

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

4 participants