Skip to content

GradientSimilarity explanations images.#695

Merged
RobertSamoilescu merged 6 commits intoSeldonIO:masterfrom
RobertSamoilescu:similarity-imgs
Jun 10, 2022
Merged

GradientSimilarity explanations images.#695
RobertSamoilescu merged 6 commits intoSeldonIO:masterfrom
RobertSamoilescu:similarity-imgs

Conversation

@RobertSamoilescu
Copy link
Contributor

This PR adds two images in the method description for the GradientSimilarity.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@RobertSamoilescu RobertSamoilescu requested a review from jklaise June 9, 2022 13:14
@codecov
Copy link

codecov bot commented Jun 9, 2022

Codecov Report

Merging #695 (5ba8df9) into master (dc0e6ec) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #695   +/-   ##
=======================================
  Coverage   80.55%   80.55%           
=======================================
  Files         105      105           
  Lines       11790    11790           
=======================================
  Hits         9497     9497           
  Misses       2293     2293           

Copy link
Contributor

@jklaise jklaise left a comment

Choose a reason for hiding this comment

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

Looks good! A couple of minor suggestions:

  1. The figure captions could be made more obvious that they're captions, e.g. by using italics
  2. could you remove the [source] link at the top as it's not leading anywhere?

@RobertSamoilescu RobertSamoilescu merged commit e5814a8 into SeldonIO:master Jun 10, 2022
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.

2 participants