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

System-wise detection training error and fix #98

Open
Jeroenvanwely opened this issue Jun 8, 2023 · 0 comments
Open

System-wise detection training error and fix #98

Jeroenvanwely opened this issue Jun 8, 2023 · 0 comments

Comments

@Jeroenvanwely
Copy link

I noticed the following issue when trying to train and evaluate the system-wise detection model. When trying to train using fit(), it returns an error:
TypeError: set_training_data() got an unexpected keyword argument 'inputs'.
When the model is initialized, it goes through SystemWiseDetection_skinterface.py. Here the BaseSKI class is initialised as a parent class. In the BaseSKI, we can see the following code:
1
The red box outlines how fit_available, predict_available, predict_score_available and produce_available are defined based on the presence of the function in the primitive class (in this case, the SystemWiseDetectionPrimitive class). This class does not contain the fit() function and, therefore, self.fit_available on line 25 is set to False (which is correct). However, after the initialisation of the BaseSKI, we return to SystemWiseDetection_skinterface.py:
2
Where the red box outlines a redefining/overwriting of fit_available, predict_available and produce_available, where fit_available is now changed to True even though this function does not exist in the SystemWiseDetectionPrimitive class.

This causes the error to occur. Moreover, I noticed all the *_skinterface.py files contain this overwriting of fit_available, predict_available and produce_available, which would seem to make the initial definitions in the BaseSKI class init redundant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant