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

Spatial template use shows template bin structure now in the simulation #148

Open
CristianAntochi opened this issue Jun 30, 2021 · 1 comment

Comments

@CristianAntochi
Copy link
Collaborator

CristianAntochi commented Jun 30, 2021

Hello all,
Back in February the use of the spatial templates for spatial dependent sources was a bit more complicated than now, but the FD simulation of the source using the template did not show bin structures (say r, binned uniformly in r^2), plot for reference:
wall_r_z-5

Now it does!

image

@CristianAntochi CristianAntochi added the bug Something isn't working label Jun 30, 2021
@JelleAalbers
Copy link
Member

Interesting Cristian. Indeed, your two figures are clearly different: the second seems to have way more events overall, especially at higher r, and fewer events at lower r. The higher density causes the bin structure to appear more prominent -- whether that's the only reason why the bin structure appears more prominent or whether there is something else going on as well is hard to say.

The change you're referring to between February and now is #100. As described there, in February, you had to pass some funnily normalized histogram that was neither a PDF nor the expected events/bin. Since March, you have to pass an events/bin histogram.

There are fifteen possible explanations for the difference you're seeing, corresponding to the yes/no answers to the four questions below -- except 'yes to all', which is ruled out because the plots are different.

  1. Did you give flamedisx the template correctly in February?
  2. Are you giving flamedisx the template correctly now?
  3. Did flamedisx work correctly in February?
  4. Does flamedisx work correctly now?

So I don't think we can say there is a flamedisx bug now just yet. It's possible, but so are other explanations. For example, if you are giving exactly the same template to your code without changing your normalization, the answer to either 1 or 2 is 'No' because the convention on what to pass has changed (see above).

You probably already saw the notebook at https://github.com/FlamTeam/flamedisx-notebooks/blob/master/spatial_variation_demo.ipynb, but if not, that might help.

@JelleAalbers JelleAalbers removed the bug Something isn't working label Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants