-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
mlr3 targets post #41
base: main
Are you sure you want to change the base?
Conversation
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.
I don't really know targets so can only really comment once I am familiar
knitr::opts_chunk$set(echo = FALSE) | ||
``` | ||
|
||
The [mlr3-learndrake](https://github.com/mlr-org/mlr3-learndrake) repository has been deprecated and (finally) been ported to it's successor {targets}. |
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.
it's -> its
• built branch bm_be2cba98 | ||
✔ skip branch bm_da5f38fa | ||
• built pattern bm | ||
``` |
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.
Maybe add one of these figures with red and green targets instead of the log.
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.
It tried but I couldn't get them large enough to be well visible in the post. Usually one does it with zooming but this is ofc not an option here.
Blocked by #43 |
If we merge to quarto, we could use a mermaid diagram for the visualization |
dcc2a4a
to
166c415
Compare
f10f1ca
to
2b02238
Compare
38defd7
to
6df4fdb
Compare
An image would be nice but the output from
tar_visNetwork()
is too large and the nodes become too small.Thoughts? Feedback?