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.

Sign up for GitHub

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

Jump to bottom

Enable specification of options in a PredictionObservableComponent #834

Closed
dvandyk opened this issue Apr 15, 2024 · 2 comments · Fixed by #849
Closed

Enable specification of options in a PredictionObservableComponent #834

dvandyk opened this issue Apr 15, 2024 · 2 comments · Fixed by #849
Assignees
@MJKirk
Labels
bug Genuine error in the software
Milestone
Release 1.1

Comments

@dvandyk
Copy link
Member

dvandyk commented Apr 15, 2024

No description provided.

@mreboud mreboud added the bug Genuine error in the software label Apr 23, 2024
@mreboud mreboud added this to the Release 1.1 milestone Apr 23, 2024
@MJKirk
Copy link
Contributor

MJKirk commented May 7, 2024

I'm guessing this should just take a dict to specify options for an observable, e.g. the following would be the correct valid yaml:

predictions:
  - name: B-to-l-nu
    observables:
      - name: B_u->lnu::BR
        options: { l : tau }
      - name: B_u->lnu:BR
        options: { l : mu }

?

@dvandyk
Copy link
Member Author

dvandyk commented May 9, 2024

Yes please!

@MJKirk MJKirk mentioned this issue May 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MJKirk MJKirk

Labels
bug Genuine error in the software
Projects
None yet
Milestone
Release 1.1
Development

Successfully merging a pull request may close this issue.

Allow specifying options to a prediction observable in an AnalysisFile MJKirk/eos
3 participants
@dvandyk @mreboud @MJKirk