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

Suspected coding mistake in PerformanceEvaluator #202

Closed
prabaey opened this issue Jun 2, 2023 · 1 comment · Fixed by #203
Closed

Suspected coding mistake in PerformanceEvaluator #202

prabaey opened this issue Jun 2, 2023 · 1 comment · Fixed by #203
Labels
bug Something isn't working

Comments

@prabaey
Copy link

prabaey commented Jun 2, 2023

Description

There seems to be a coding mistake in the function _evaluate_performance_regression within the class PerformanceEvaluator. I think I spotted a typo in the transformation of the training and test set to numpy arrays, which leads to the estimator being trained and tested on the same data.

How to Reproduce

Simply investigate the source code for the function _evaluate_performance_regression within the class PerformanceEvaluator in the file eval_performance.py.

Expected Behavior

The following snippet of code

 X_train = np.asarray(X_test)
 X_test = np.asarray(X_test)
 y_train = np.asarray(y_test)
 y_test = np.asarray(y_test)

should be changed to:

 X_train = np.asarray(X_train)
 X_test = np.asarray(X_test)
 y_train = np.asarray(y_train)
 y_test = np.asarray(y_test)
@robsdavis
Copy link
Contributor

@prabaey, thanks for posting this issue! It does indeed seem to be a mistake. A PR to fix this will be created shortly.

@robsdavis robsdavis linked a pull request Jun 2, 2023 that will close this issue
4 tasks
@robsdavis robsdavis added the bug Something isn't working label Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants