-
Notifications
You must be signed in to change notification settings - Fork 3
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
TypeError: unsupported operand type(s) for |: 'NoneType' and 'NoneType' #15
Comments
Hi, thanks for reporting this! Darts should be an optional dependency, but it looks like there’s an issue with that. As a workaround you can pip install darts, but we’ll address the issue with a new release today. |
I released a hotfix that should address the issue. We'll add more thorough testing later. Could you check if v0.3.1 solves the issue for you? Thank you! |
Update: also added tests with #17 to ensure that we don't inadvertently depend on optional dependencies in the future. |
Hi Laurent. It works now! Thank you. BTW, is XGBoost the only regressor that it can use? Can it use CatBoost, or LightGBM? I want to perform KFold cross validation and hyperparameter tuning during the training stage. Is there a pipeline for this? Any suggestions are welcome. Please keep up the great work. This library is awesome. |
You can use any scikit-learn compatible regressor you like, including CatBoost and LightGBM, yes. We'll try to make that more clear in the README.
Which hyperparameters do you want to train? I'll see if I can add some examples. |
Actually, thinking about it, I could just do the Hyperparameter tuning with cross-validation and pick the best estimator based on the best parameters. Then I would feed the best estimator into 'ConformalCoherentQuantileRegressor'. I think that should work. BTW, how do I change the size of the calibration set or is this already optimized? |
Yes, that should work well. Note that an upcoming release of Conformal Tights should decouple the wrapped estimator entirely, allowing it to benefit from the full training data set without the calibration set affecting its performance!
You can change the size of the calibration set when constructing the ConformalCoherentQuantileRegressor as follows: conformal_predictor = ConformalCoherentQuantileRegressor(conformal_calibration_size=(0.3, 1440)) With that setting, it will use 30% of the training data or 1440 samples (whichever is smaller). These are the trade-offs:
|
Thanks for the quick response, and I apologize for the late response. It's good to hear about features for the upcoming release. I am currently using this library for a client, and they are interested in getting quantiles and a PDF. Will future releases also include PDFs? Yes, predicting the quantile and intervals does take time. I wonder if this can be sped up via a GPU. However, I think this is ML library-dependent. |
Hi there. I am trying to run the regression example but I when I try to load the library below
from conformal_tights import ConformalCoherentQuantileRegressor
I get the error
Not sure why it is trying to load the DartsForecaster here. Any suggestions? Thanks.
~ C
The text was updated successfully, but these errors were encountered: