Skip to content

rerun daemonflux example notebook#883

Merged
marialiubarska merged 1 commit into
masterfrom
daemon_example
Oct 27, 2025
Merged

rerun daemonflux example notebook#883
marialiubarska merged 1 commit into
masterfrom
daemon_example

Conversation

@marialiubarska
Copy link
Copy Markdown
Contributor

updated the example notebook after bugfix

Copy link
Copy Markdown
Collaborator

@JanWeldert JanWeldert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could delete some of the larger outputs to reduce the file size a bit (the diff doesn't show up in the online version), but apart from that it looks good.

@marialiubarska
Copy link
Copy Markdown
Contributor Author

Ah, I didn't notice that. I think it would be nice to keep the plots with differences vs Honda et al. I copied the plots to google slides: https://docs.google.com/presentation/d/1XSUcnCEL3wXcfXiWMZXQCg9NUIb5hE0GFxD5tF5KBKg/edit?usp=sharing

@marialiubarska marialiubarska merged commit d31297f into master Oct 27, 2025
2 checks passed
@marialiubarska marialiubarska deleted the daemon_example branch October 29, 2025 00:05
@thehrh
Copy link
Copy Markdown
Collaborator

thehrh commented May 20, 2026

@marialiubarska @JanWeldert Did both of you verify the notebook outputs (https://github.com/icecube/pisa/blob/db24838f292d1f31a7d5a192c239e2e1f6318b44/pisa_examples/test_daemonflux_stage.ipynb) here?

3 years of DRAGON should not have summed (neutrino) count expectations limited to ~160, but rather ~700. And indeed, when I run the notebook on the current master branch, the maximum bin count is around that level. (Or compare the IceCube_3y_oscillations_example.ipynb notebook.)

Even my daemonflux/Honda ratio map looks different:
image

In general, no settings should have changed when it comes to the public data sample, which is used by this notebook, and neither should the daemon_flux/honda_ip services have undergone any changes affecting their default physics outputs, which makes me wonder how the results in this PR were produced.

I have a modified version of this notebook in the works in #930 (so no need for a dedicated PR fixing this).

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

Successfully merging this pull request may close these issues.

3 participants