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

Learning policy fix #246

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Learning policy fix #246

wants to merge 7 commits into from

Conversation

maypink
Copy link
Collaborator

@maypink maypink commented Dec 7, 2023

No description provided.

@maypink maypink linked an issue Dec 7, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Dec 9, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (46b53a4) 71.94% compared to head (66f3014) 71.91%.
Report is 2 commits behind head on main.

Files Patch % Lines
...misers/adaptive/mab_agents/contextual_mab_agent.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #246      +/-   ##
==========================================
- Coverage   71.94%   71.91%   -0.03%     
==========================================
  Files         136      136              
  Lines        8127     8147      +20     
==========================================
+ Hits         5847     5859      +12     
- Misses       2280     2288       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pep8speaks
Copy link

pep8speaks commented Dec 14, 2023

Hello @maypink! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 4:1: F401 'matplotlib.pyplot as plt' imported but unused
Line 5:1: F401 'scipy.stats.mannwhitneyu' imported but unused
Line 5:1: F401 'scipy.stats.kruskal' imported but unused
Line 5:1: F401 'scipy.stats.ttest_ind' imported but unused
Line 8:1: F401 'golem.core.paths.project_root' imported but unused

Line 234:121: E501 line too long (124 > 120 characters)
Line 294:121: E501 line too long (121 > 120 characters)

Comment last updated at 2023-12-17 15:46:53 UTC

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.

Learning Policy fix
3 participants