CliMA / CliMA/CalibrateEmulateSample.jl

Catch-all error can be quite misleading

Open
#379 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Julia
Stars
90
Forks
16
PR merge metrics
No merged PRs in 30d

Description

## issue

This error message:

https://github.com/CliMA/CalibrateEmulateSample.jl/blob/101d3f6ece288e3d56f2fa51e829709f40f87e6b/src/Emulator.jl#L40-L51

designed to catch unknown or unimplemented machine learnign tools can be misleading. Primarily, if one gets the data-type incorrect for the `inputs` in say `predict(mlt, inputs...)` it throws the error implying that the `mlt` argument was incorrect!

Should be corrected to a more specific message, and not thrown in these cases to avoid confusion.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.