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.

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

Decouple IndexListView from ExpectedLossWidget #101

Open
genedan opened this issue Jul 14, 2023 · 0 comments
Open

Decouple IndexListView from ExpectedLossWidget #101

genedan opened this issue Jul 14, 2023 · 0 comments
Labels
refactor

Comments

@genedan
Copy link
Collaborator

genedan commented Jul 14, 2023 •
edited

The IndexListView is dependent on the existence of an ExpectedLossWidget. We cannot test these two classes independently of each other, but we should. This is because the IndexListView should be available for other models to use.

The current test that would need to be updated after this change would be test_index.py.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@genedan