Skip to content

SDML does not work in sandwich example #138

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.

Sign up for GitHub

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

Jump to bottom
Closed
bellet opened this issue Dec 20, 2018 · 2 comments
Closed

SDML does not work in sandwich example #138

bellet opened this issue Dec 20, 2018 · 2 comments
Milestone
v0.5.0

Comments

@bellet
Copy link
Member

bellet commented Dec 20, 2018

Description

SDML does not seem to do anything in the sandwich example from the doc (see figure below). We should make sure it is due to the default hyperparameters (and fix them in the example is that's the case).

sphx_glr_plot_sandwich_001
@bellet bellet added this to the v0.5.0 milestone Dec 20, 2018
@wdevazelhes
Copy link
Member

In a previous version of metric-learn, scipy, and numpy, the example seemed to work. For instance, I have this result for theses versions:
Linux-4.4.0-141-generic-x86_64-with-debian-stretch-sid
('Python', '2.7.15 |Anaconda, Inc.| (default, Dec 14 2018, 19:04:19) \n[GCC 7.3.0]')
('NumPy', '1.11.3')
('SciPy', '0.17.1')
('Scikit-Learn', '0.17.1')
metric-learn: from commit ddfac99

figure_1-2

I guess there is some problem coming from new versions of either metric-learn, scikit-learn, scipy or numpy...

@wdevazelhes
Copy link
Member

wdevazelhes commented Jan 11, 2019 •
edited
Loading

Probably related to #154

@wdevazelhes wdevazelhes mentioned this issue Mar 18, 2019
Merged
5 tasks
@bellet bellet mentioned this issue Mar 26, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.5.0
Development

No branches or pull requests
2 participants
@bellet @wdevazelhes