-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Bug fixes for performance over time calculations #1602
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
Conversation
new file: Bugs MWE.ipynb
modified: Bugs MWE.ipynb
modified: autosklearn/automl.py modified: autosklearn/automl_common (modified content)
modified: autosklearn/automl_common (modified content)
Heyo, thanks for the contribution :) I've run the tests and hopefully they will all pass. I will do a pass and leave a review once I can! |
All the errors seem to have no relation to the files I have modified. I have rerun |
Hi, I'm a new user and was wondering: is there an explanation of what each of these curves within the |
@carlosnatalino, as far as I understand,
|
Hi @nklemashev thank you very much for the explanation. Very appreciated! |
Hi @nklemashev, Apologies for the delay, I was on vacation, I reran the tests and will look at this either today or tomorrow! Best, |
I ran into the same issue in PR #1606. The error likely occures because pytest-xdist exported some functionality to another package called pytest-forked so the --forked flag does not get recognized anymore. This should be fixed soon however, and thereafter the failed tests should pass. |
As an update, I am now half-back and pushing to update scikit-learn and some of internal dependancies before moving onto these PR's. Apologies for delays and your PR is definitely appreciated. |
I have found and corrected several bugs in getting performance over time:
All these four features may be seen in the following MWE:
The output of this script is
A nicer check for these issues with graphics is presented in Jupyter notebook: Bugs MWE.
The same notebook with corrected autosklearn code: Bugs MWE Corrected.
I have slightly modified code in performance_over_time_ method of AutoML class for fixing these issues.
P.S.: Also modified Makefile to prevent black command from reformatting everything in the directory, in which autosklearn is located. I followed your contribution guidelines and created Python virtual environment directory my-virtual-env in the same directory as autosklearn and running black autosklearn/.* reformats all Python packages in my-virtual-env.