-
Notifications
You must be signed in to change notification settings - Fork 192
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
adding weights option to fill_events #4458
Conversation
Signed-off-by: Maxime Regeard <regeard@apc.in2p3.fr>
Signed-off-by: Maxime Regeard <regeard@apc.in2p3.fr>
Codecov Report
@@ Coverage Diff @@
## main #4458 +/- ##
==========================================
+ Coverage 94.87% 94.93% +0.05%
==========================================
Files 216 218 +2
Lines 30626 30888 +262
==========================================
+ Hits 29057 29322 +265
+ Misses 1569 1566 -3
... and 45 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Thanks @MRegeard! I'm not sure we would really like to expose this, where we explicitly refer to "events" (see my comment ). What do you think @registerrier ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MRegeard! Please add a minimal test to establish the behavior...
Signed-off-by: Maxime Regeard <regeard@apc.in2p3.fr>
Signed-off-by: Maxime Regeard <regeard@apc.in2p3.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MRegeard . Looks good. Please a a test for this new functionality.
Signed-off-by: Maxime Regeard <regeard@apc.in2p3.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MRegeard . No further comment.
This pull request is related to #4457.
I added a weights option to
Map.fill_events
. SinceMap.fill_events
usesMap.fill_by_coord
which has a weights option, the implementation is straight forward.