Removing DLPipelines.jl; Learning method -> Learning task #198
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This will incorporate DLPipelines.jl directly into FastAI.jl and drop the dependency. This is a BREAKING change.
In the process, the following changes will be made to the
LearningMethod
interfaces:LearningMethod
toLearningTask
,method
totask
LearningTask
was an abstract parametric type forLearningMethod
s to signify that there can be multiple ways of solving a task, but this was dropped at some point.LearningTask
is imo a more intention-revealing name, as method is a very ambiguous term (especially in Julia-land). This is especially true for the short variable nametask
vsmethod
.LearningMethod
will also be renamed. Specifically,[Abstract]BlockMethod -> [Abstract]BlockTask
andSupervisedMethod -> SupervisedTask
.methoddataset -> taskdataset
andmethoddataloaders -> taskdataloaders
findlearningmethods -> findlearningtasks
LearningMethod
interface:encode(method, ctx, sample)
toencodesample
. This way,encode
is reserved forEncoding
s.decodeŷ
todecodeypred
. Easier to write and to differentiate fromdecodey
. Old name will be aliased.