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

Pin skorch version to below 0.14 #465

Merged
merged 3 commits into from Aug 18, 2023

Conversation

ZamboniMarco99
Copy link
Contributor

See #462

Updated environment.yml and requirements.txt to pin skorch package version to less than 0.14. This change ensures compatibility with existing code which may not support the latest version of skorch. For example it expects classifiers like EEGClassifier to have the 'classes_inferred' attribute.
@ZamboniMarco99 ZamboniMarco99 marked this pull request as draft August 18, 2023 09:22
@ZamboniMarco99 ZamboniMarco99 marked this pull request as ready for review August 18, 2023 09:23
@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

Merging #465 (1269ae3) into master (2335a4e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #465   +/-   ##
=======================================
  Coverage   84.13%   84.13%           
=======================================
  Files          59       59           
  Lines        4267     4267           
=======================================
  Hits         3590     3590           
  Misses        677      677           

@bruAristimunha bruAristimunha merged commit 31cfd6c into braindecode:master Aug 18, 2023
6 checks passed
@bruAristimunha
Copy link
Collaborator

Thank you so much @ZamboniMarco99 :) nice first PR

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

Successfully merging this pull request may close these issues.

None yet

2 participants