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.

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

[ENH] GPFA with continuous data #539

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Moritz-Alexander-Kern
Copy link
Member

@Moritz-Alexander-Kern Moritz-Alexander-Kern commented Feb 10, 2023

This pull request extends the functionality of the GPFA classes methods to accept continuous data.

The changes are done to the methods fit, transform and fit_transform, which now accept continuous data.

Unit tests and docstrings have been written to ensure that the changes are properly implemented.

This PR seeks a continuation of the previously started development in PR #507.

The current input for continuous data to fit, transform and fit_transform is a numpy recarray

Todo

  • decide on input type for continuous data
  • integrate with trial object

Co-authored-by: Jonah Pearl [email protected]

* Allow seqs to be passed directly to gpfa fit

* Small fix to error checking

* Typo

* another typo

* fixed formatting for pep8

* fixed docstring of transform after introducing error while resolving merge conflicts

* added two unit-test

* add more unit-tests

* remove blank line

---------

Co-authored-by: Jonah Pearl <[email protected]>
@Moritz-Alexander-Kern Moritz-Alexander-Kern added the enhancement Editing an existing module, improving something label Feb 10, 2023
@coveralls
Copy link
Collaborator

coveralls commented Feb 10, 2023

Coverage Status

Coverage: 87.455% (-0.04%) from 87.493% when pulling 5b966ab on INM-6:enh/GPFA_continous_data into 0896873 on NeuralEnsemble:master.

@Moritz-Alexander-Kern Moritz-Alexander-Kern changed the title ENH/ GPFA with continous data ENH/ GPFA with continuous data Feb 10, 2023
@Moritz-Alexander-Kern Moritz-Alexander-Kern changed the title ENH/ GPFA with continuous data [ENH] GPFA with continuous data Nov 27, 2023
@Moritz-Alexander-Kern Moritz-Alexander-Kern added this to the v1.3.0 milestone Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Editing an existing module, improving something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants