scikit-fda
scikit-fda copied to clipboard
Feature/fdm
- Added FDM feature in the dimensionality reduction submodule of the preprocessing module
_fdm.py. - Included a test file for testing the new method
test_fdm.py. - Created and added an example for the wiki
plot_fdm.py.
Ignoring flake8 errors WPS440 and WPS441 in the doc example file plot_fdm .
Again, sorry for taking so long reviewing this. The main functionality is great, I just have a couple of nitpicks in documentation and tests.
If you do not have the time/will to do the changes, just tell me and I will apply them myself (but please, answer my question about the comparison in the tests, as I truly do not remember which was the reference package).
I implemented and pushed all changes.
There is a problem with the style tests FileNotFoundError: [Errno 2] No such file or directory: 'examples/plot_fdm.py skfda/preprocessing/dim_reduction/__init__.py .
Also, I created a web version of the study that I did on the FDM method, FDM GitHub pages I was thinking it could be interesting to reference it in a note in the FDM class example of use. The website is essentially a more in depth version of the example of use notebook, including mathematical background and further experiments. Please let me know your view on this.
Also, I created a web version of the study that I did on the FDM method, FDM GitHub pages I was thinking it could be interesting to reference it in a note in the FDM class example of use. The website is essentially a more in depth version of the example of use notebook, including mathematical background and further experiments. Please let me know your view on this.
Hey, that is cool! Yes, you can add a note on the top pointing to that page, or just reference it with a footnote cite, also at the top of the page, so that nobody misses it.
Thanks for the comments and suggestions. I will work on it in two weeks time, right now I'll be unable.
Sure! Take the time you need.
At the end I found some free time and did the changes myself, as they where small. Thanks again for your contribution, I wish you luck with your current endeavors!
@all-contributors add @E105D104U125 for code, documentation, tests, examples and research.
At the end I found some free time and did the changes myself, as they where small. Thanks again for your contribution, I wish you luck with your current endeavors!
Thanks for finishing things up @vnmabus. I wish you luck too!