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
Add output parser for dssim algo. #33
Conversation
src/clj/kamera/core.clj
Outdated
@@ -147,7 +151,7 @@ | |||
(.getAbsolutePath actual) | |||
(.getAbsolutePath difference)] | |||
opts) | |||
mean-absolute-error (when-let [e (last (re-find #"all: .* \((.*)\)" stderr))] | |||
mean-absolute-error (when-let [e ((compare-parsers metric) stderr)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a good idea to have a default too, so that we're not limiting the user to just the enumerated algos. Not sure which one would be better though, I guess the mae one would be best because people may already be using that one now with other algos.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also this was my oversight, but could we also rename the mean-average-error
to be metric
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok 👍
Fixes #32 (as combination of dssim and no :crop will address that in config options). |
Looks great @jleonard-r7 thanks, are you happy with how it works? I'll merge if so. |
Yep, it’s ready to merge. |
Co-authored-by: Jonathan Leonard <johanatan@gmail.com>
Resolves #32