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] Integrate trials object with GPFA #610

Merged
merged 56 commits into from
Mar 26, 2024

Conversation

Moritz-Alexander-Kern
Copy link
Member

This pull request introduces an enhancement to GPFA using the the elephant.trials module to represent trials. This change is part of a greater effort to provide a more structured and unified approach for handling trial data in Elephant.

Trial representations

@pep8speaks
Copy link

pep8speaks commented Dec 11, 2023

Hello @Moritz-Alexander-Kern! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-03-26 11:21:07 UTC

@Moritz-Alexander-Kern Moritz-Alexander-Kern changed the title [Feature] Integrate trials object with GPFA [ENH] Integrate trials object with GPFA Dec 11, 2023
@coveralls
Copy link
Collaborator

coveralls commented Dec 11, 2023

Coverage Status

coverage: 87.935% (-0.3%) from 88.264%
when pulling f993f8c on INM-6:feature/trials_gpfa
into 6363690 on NeuralEnsemble:master.

@Moritz-Alexander-Kern Moritz-Alexander-Kern added this to the v1.1.0 milestone Jan 17, 2024
Copy link
Collaborator

@kohlerca kohlerca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The functionality to make the conversion looks good to me. I made some small suggestions to improve the code and docstrings.

elephant/utils.py Outdated Show resolved Hide resolved
elephant/utils.py Outdated Show resolved Hide resolved
elephant/utils.py Outdated Show resolved Hide resolved
elephant/utils.py Outdated Show resolved Hide resolved
elephant/utils.py Outdated Show resolved Hide resolved
elephant/utils.py Outdated Show resolved Hide resolved
Copy link
Member Author

@Moritz-Alexander-Kern Moritz-Alexander-Kern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build wheels

@Moritz-Alexander-Kern Moritz-Alexander-Kern merged commit 9326804 into NeuralEnsemble:master Mar 26, 2024
15 checks passed
@Moritz-Alexander-Kern Moritz-Alexander-Kern deleted the feature/trials_gpfa branch October 9, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants