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

latrendBatch output error for errorHandling = "pass" #46

Closed
3 tasks done
niekdt opened this issue Jul 20, 2021 · 0 comments
Closed
3 tasks done

latrendBatch output error for errorHandling = "pass" #46

niekdt opened this issue Jul 20, 2021 · 0 comments
Assignees
@niekdt
Labels
bug Something isn't working
Milestone
1.2.0

Comments

@niekdt
Copy link
Collaborator

niekdt commented Jul 20, 2021 •
edited
Loading

latrendBatch() fails when one or more methods throw an error during fitting, and errorHandling = "pass". This is because the function attempts to convert the output to lcModels, but the list might contain simpleError objects.

  • Handle this case. Returning a list in case of errors.
  • Improve output messages in case of errors
  • Add tests
@niekdt niekdt added the bug Something isn't working label Jul 20, 2021
@niekdt niekdt added this to the 1.2.0 milestone Jul 20, 2021
@niekdt niekdt self-assigned this Jul 20, 2021
niekdt added a commit that referenced this issue Jul 20, 2021
@niekdt
Handle error handling cases in latrendBatch(). Added tests. (#46)
7e7aae8
@niekdt niekdt closed this as completed Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niekdt niekdt

Labels
bug Something isn't working
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@niekdt