Skip to content
This repository has been archived by the owner on Nov 29, 2022. It is now read-only.

Fix check of rho argument to sim_pte #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix check of rho argument to sim_pte #38

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion pylift/generate_data.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ def sim_pte(N=1000, n_features=20, beta=None, rho=0, sigma=np.sqrt(2), beta_den=
if p < 4:
raise ValueError("uplift: The number predictors must be equal or greater than 4")

if rho < 0 | rho > 1:
if rho < 0 or rho > 1:
raise ValueError("uplift: rho must be between 0 and 1")

if sigma < 0:
Expand Down