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 MIDA, CoIRLS, distribution plot, and brain example #246
Conversation
@sz144 I hope to work with you to resolve and merge this PR after merging #284. Share the issues remaining with the team today. |
examples/autism_detection/config.py
Outdated
@@ -0,0 +1,39 @@ | |||
""" | |||
Default configurations for cardiac MRI data (ShefPAH) processing and classification |
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.
Have you checked your code/comment? Are you sure this config is about cardiac? The filename says autism.
examples/autism_detection/config.py
Outdated
# ----------------------------------------------------------------------------- | ||
_C.DATASET = CN() | ||
_C.DATASET.ROOT = "../data" | ||
_C.DATASET.PIPELINE = "cpac" |
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.
Any reference to help users to read further? Particularly, available options.
_C.DATASET = CN() | ||
_C.DATASET.ROOT = "../data" | ||
_C.DATASET.PIPELINE = "cpac" | ||
_C.DATASET.ATLAS = "rois_cc200" |
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.
Any reference (as a comment)?
Possible to provide (as a comment) available options?
from kale.pipeline.multi_domain_adapter import _CoIRLS | ||
|
||
|
||
def plot_scores_dist(ys_score, yt_score, title=None): |
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.
Is this reusable for problems of the same nature? If yes, then it should go to core API, e.g. visualization.
examples/video_loading/README.md
Outdated
- [4. Alternate Video Frame Sampling Methods](#4-alternate-video-frame-sampling-methods) | ||
- [5. Using VideoFrameDataset for training](#5-using-videoframedataset-for-training) | ||
- [6. Allowing Multiple Labels per Sample](#6-allowing-multiple-labels-per-sample) | ||
- [5. Conclusion](#5-conclusion) |
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.
Why change this and with two 5s and 6s?
kale/embed/decomposition.py
Outdated
Haiping Lu, K.N. Plataniotis, and A.N. Venetsanopoulos, "MPCA: Multilinear Principal Component Analysis of Tensor | ||
Objects", IEEE Transactions on Neural Networks, Vol. 19, No. 1, Page: 18-39, January 2008. For initial Matlab | ||
implementation, please go to https://uk.mathworks.com/matlabcentral/fileexchange/26168. | ||
"""Python implementation of tensor decomposition approach Multilinear Principal Component Analysis (MPCA) |
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.
As said, factorization should be better than decomposition.
Also, have you updated the docs?
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.
Great job.
Only some minor issues remain to be fixed (see comments). Once done, it can be merged. Many thanks.
Fixes #173.
Description
kale.embed.mpca
->kale.embed.factorization
kale.embde.factorization.MIDA
kale.interpret.visualize.distplot_1d
kale.pipeline.multi_domain_adapter.CoIRLS
Status
Ready
Types of changes
docs
manually updated for new API.