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

add decision utility funciton draft #477

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

OriolAbril
Copy link
Member

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@drbenvincent
Copy link
Contributor

  • Looks great. I guess there will be some more explanation text added in the final draft?
  • There's potentially scope for doing a simple model 1 with just one outcome variable as a really simple start, then bring in the model with 3 outcome variables to demonstrate the generality.
  • Could be worth describing where the utility function comes from, or that this is up to what the person finds most important.

@OriolAbril
Copy link
Member Author

I guess there will be some more explanation text added in the final draft?

Yes, this is only getting the code to work and sharing in case someone had suggestions on the model. I'll try to come back to this and add the explanation and finish t some point next week

@fonnesbeck
Copy link
Member

Dead or alive?

@OriolAbril
Copy link
Member Author

Probably best to consider it dead. I do still have it on the TODO but with many higher priority things before it, I don't think I'll get to it anytime soon. It is also ok with me if someone wants to take over and finish it too.

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