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 SapientML to automl benchmark #630

Merged
merged 3 commits into from
Nov 15, 2024
Merged

Conversation

kimusaku
Copy link
Contributor

SapientML is an AutoML technology that can learn from a corpus of existing datasets and their human-written pipelines, and efficiently generate a high-quality pipeline for a predictive task on a new dataset.

Signed-off-by: Kosaku Kimura <kimura.kosaku@fujitsu.com>
Copy link
Collaborator

@PGijsbers PGijsbers left a comment

Choose a reason for hiding this comment

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

Thanks for your contributions! I haven't had time to try this out yet, but I do already have a couple questions and suggested changes based on the PR. Please have a look at them.

frameworks/SapientML/exec.py Show resolved Hide resolved
frameworks/SapientML/requirements.txt Outdated Show resolved Hide resolved
frameworks/SapientML/requirements.txt Outdated Show resolved Hide resolved
frameworks/SapientML/setup.sh Show resolved Hide resolved
resources/config.yaml Outdated Show resolved Hide resolved
@PGijsbers PGijsbers added the framework add For issues with a framework to be added label Aug 30, 2024
@PGijsbers
Copy link
Collaborator

@kimusaku Are you still planning to work on this?

@PGijsbers PGijsbers added the Stale Last response by user was long ago label Oct 5, 2024
@kimusaku
Copy link
Contributor Author

kimusaku commented Oct 6, 2024

@PGijsbers Yes, we are still working on this, especially in making SapientML Python 3.9-compatible. We will be able to update the PR soon.

@PGijsbers PGijsbers removed the Stale Last response by user was long ago label Oct 6, 2024
* WIP

Signed-off-by: Kosaku Kimura <kimura.kosaku@fujitsu.com>

* Issue fix in setup.sh for SapientML

* Updation of config file as Sapientml supports Python version 3.9

Signed-off-by: HimanshuRRai <himanshu.rai@fujitsu.com>

---------

Signed-off-by: Kosaku Kimura <kimura.kosaku@fujitsu.com>
Signed-off-by: HimanshuRRai <himanshu.rai@fujitsu.com>
Co-authored-by: muhammed-nafi-k-a <muhammednafi.a@fujitsu.com>
Co-authored-by: HimanshuRRai <himanshu.rai@fujitsu.com>
Signed-off-by: HimanshuRRai <himanshu.rai@fujitsu.com>
Copy link
Collaborator

@PGijsbers PGijsbers left a comment

Choose a reason for hiding this comment

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

Very sorry for the slow reply. I finally had a chance to run it locally and it seems to work fine... except when running the validation benchmark (python runbenchmark.py sapientml validation -f 0):

Summing up scores for current run:
                 id                    task  fold framework constraint     result      metric  duration      seed                                                                    info
  openml.org/t/9910             bioresponse     0 SapientML       test   0.797310         auc      59.1 432708598                                                                     NaN
openml.org/t/125920           dresses-sales     0 SapientML       test   0.597701         auc      19.3 432708598                                                                     NaN
  openml.org/t/2079              eucalyptus     0 SapientML       test -11.689800 neg_logloss      20.6 432708598                                                                     NaN
openml.org/t/167125 internet-advertisements     0 SapientML       test   0.929463         auc      59.0 432708598                                                                     NaN
  openml.org/t/9950              micro-mass     0 SapientML       test  -1.242880 neg_logloss      35.2 432708598                                                                     NaN
  openml.org/t/3917                     kc1     0 SapientML       test   0.720932         auc      23.5 432708598                                                                     NaN
openml.org/t/168868              APSFailure     0 SapientML       test   0.897341         auc     206.1 432708598                                                                     NaN
openml.org/t/168877           diabetes130US     0 SapientML       test        NaN neg_logloss      65.8 432708598 NoResultError: All candidate scripts failed. Final script is not saved.
openml.org/t/211985           census-income     0 SapientML       test   0.709671         auc      92.7 432708598

The scripts generated can be found here:https://gist.github.com/PGijsbers/ed825dab98b85ab43b7bbbc00781b694

I could merge it (sapient is not part of CI currently), but perhaps you'd prefer to fix it first? If it's reproducible for you.

@kimusaku
Copy link
Contributor Author

@PGijsbers thanks so much. We have fixed the issue, and now this PR can be merged.

@PGijsbers PGijsbers merged commit ac09497 into openml:master Nov 15, 2024
8 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework add For issues with a framework to be added
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants