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

Fix plots #728

Merged
merged 16 commits into from
Apr 10, 2024
Merged

Fix plots #728

merged 16 commits into from
Apr 10, 2024

Conversation

SarahAlidoost
Copy link
Contributor

@SarahAlidoost SarahAlidoost commented Mar 19, 2024

closes #713

and
closes #691
closes #696
closes #206
closes #716

I think this pull request will close all issues listed above. Reviewer please check it. Thanks 😊

@SarahAlidoost SarahAlidoost added the standup Temp label- for disscussion with the team next standup label Mar 19, 2024
@SarahAlidoost SarahAlidoost removed the standup Temp label- for disscussion with the team next standup label Mar 27, 2024
@SarahAlidoost SarahAlidoost marked this pull request as ready for review March 27, 2024 09:50
@SarahAlidoost
Copy link
Contributor Author

@cwmeijer this pull request is ready for a review. I noticed an issue with text LIME in the dashboard, but it is not related to this pull request, see #738

Copy link
Contributor

@elboyran elboyran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SarahAlidoost , good work! Have some minor comments.

dianna/visualization/image.py Show resolved Hide resolved
dianna/visualization/tabular.py Show resolved Hide resolved
dianna/visualization/text.py Show resolved Hide resolved
dianna/visualization/text.py Show resolved Hide resolved
dianna/visualization/timeseries.py Show resolved Hide resolved
tests/test_dashboard.py Show resolved Hide resolved
@elboyran
Copy link
Contributor

elboyran commented Apr 8, 2024 via email

@SarahAlidoost SarahAlidoost removed the request for review from cwmeijer April 9, 2024 12:27
@SarahAlidoost
Copy link
Contributor Author

@SarahAlidoost , good work! Have some minor comments.

Thanks for reviewing, I addressed your comments. If the PR looks okay, can you approve it?

Copy link
Contributor

@elboyran elboyran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

dianna/visualization/image.py Show resolved Hide resolved
@SarahAlidoost SarahAlidoost merged commit 942be91 into main Apr 10, 2024
14 of 17 checks passed
@SarahAlidoost SarahAlidoost deleted the fix_plots branch April 10, 2024 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants