Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

model experimentation component #87

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wintonzheng
Copy link
Contributor

  • Does this PR have impact on local development experience? If yes, make sure you have a plan and add the documentations to address issues that come with the change
  • bump version
  • make a release
  • publish to pypi service

@wintonzheng wintonzheng force-pushed the model_experimentation_component branch from 859be9c to f59e194 Compare October 27, 2023 13:59
self.get_entity_id(input),
)
# TODO: validation
if treatment is None:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

question: how shall we handle the None use case?

@wintonzheng wintonzheng force-pushed the model_experimentation_component branch 2 times, most recently from e72a34a to 40fcbc2 Compare October 27, 2023 14:03
@wintonzheng wintonzheng force-pushed the model_experimentation_component branch from 40fcbc2 to 59e2e71 Compare October 27, 2023 14:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant