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

Expanded RingCounting functionality #258

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

s4294967296
Copy link
Contributor

  • Add RC data flow from boost stores, not requiring a pre-computed CNNImage file anymore
  • Add "load_from_file" config parameter. Config setting: 0/1
  • Add Execute() method in Tool class
  • Add get_next_event() method in Tool class
  • Change functionality of other methods to support both "load_from_file" settings
  • The Tool expects a CNNImage formatted input (std::vector) called "CNNImageCharge" within the "RecoEvent" boost store when not using the input from file
  • The predictions are stored in the "RecoEvent" boost store as "RingCountingSRPrediction" and "RingCountingMRPrediction"

Ringcounting/README.md:

  • Specify where the predictions are stored (RecoEvent)
  • Complete list of config parameters

- Add RC data flow from boost stores, not requiring a pre-computed CNNImage file anymore
- Add "load_from_file" config parameter. Config setting: 0/1
- Add Execute() method in Tool class
- Add get_next_event() method in Tool class
- Change functionality of other methods to support both "load_from_file" settings
- The Tool expects a CNNImage formatted input (std::vector<double>) called "CNNImageCharge" within the "RecoEvent" boost store when not using the input from file
- The predictions are stored in the "RecoEvent" boost store as "RingCountingSRPrediction" and "RingCountingMRPrediction"

Ringcounting/README.md:
- Specify where the predictions are stored (RecoEvent)
- Complete list of config parameters
@marc1uk
Copy link
Collaborator

marc1uk commented Apr 5, 2024

Hi Daniel,
Looks good from what I can follow. Would you be able to go over the updated modes in next week's software meeting on integrating this into a standard ToolChain?

@marc1uk marc1uk merged commit d348aa0 into ANNIEsoft:Application Apr 5, 2024
1 check passed
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.

2 participants