Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# ...
# Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
# license Modified BSD, see LICENSE.txt for details.
# ...
name: CI Tests and Docs
Expand All @@ -12,8 +11,12 @@ on:

jobs:
test:
name: Run Pytest Suite
name: Test Python ${{ matrix.python-version }}
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
python-version: ["3.10", "3.11", "3.12", "3.13"]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This excludes the current Python version 3.14. Is there a reason for not including 3.14?


steps:
- name: Checkout repository
Expand All @@ -22,7 +25,7 @@ jobs:
- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: '3.x'
python-version: ${{ matrix.python-version }}

- name: Install package and test dependencies
run: |
Expand All @@ -44,7 +47,7 @@ jobs:
- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: '3.x'
python-version: "3.13"

- name: Install Pandoc
run: |
Expand Down
27 changes: 27 additions & 0 deletions CITATION.cff
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
cff-version: 1.2.0
message: "If you use this software, please cite it using the metadata below."
type: software
title: "BlockInvGPFA"
abstract: >-
A scikit-learn-compatible implementation of Gaussian-process factor
analysis using block matrix inversion for efficient inference with
variable-length time series.
authors:
- family-names: "Musangu"
given-names: "Brooks M."
email: "brooks_musangu@hms.harvard.edu"
affiliation: "Department of Neurobiology, Harvard Medical School"
orcid: "https://orcid.org/0000-0002-7137-3707"
- family-names: "Drugowitsch"
given-names: "Jan"
email: "jan_drugowitsch@hms.harvard.edu"
affiliation: "Department of Neurobiology, Harvard Medical School"
orcid: "https://orcid.org/0000-0002-7846-0408"
version: "0.1.0"
license: "BSD-3-Clause"
repository-code: "https://github.com/CausalityInMotion/BlockInverseGPFA"
url: "https://blockinversegpfa.readthedocs.io/en/latest/"
keywords:
- "Gaussian Processes"
- "block matrices"
- "latent variable models"
2 changes: 1 addition & 1 deletion LICENSE.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
Code based on Elephant - Electrophysiology Analysis Toolkit: Copyright (c) 2014-2019, Elephant authors and contributors
Modifications and additions: Copyright (c) 2021 Brooks M. Musangu and Jan Drugowitsch
Modifications and additions: Copyright (c) 2021-2026 Brooks M. Musangu and Jan Drugowitsch
All rights reserved.


Expand Down
6 changes: 1 addition & 5 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,10 +82,6 @@ Tests are automatically run via [GitHub Actions](https://github.com/CausalityInM
Modified BSD License based on Elephant, see [LICENSE.txt](LICENSE.txt) for details.


## Copyright

:copyright: 2021-2025 Brooks M. Musangu and Jan Drugowitsch

## Acknowledgments

See [acknowledgments](docs/acknowledgments.rst).
See [acknowledgments](docs/acknowledgments.rst).
3 changes: 1 addition & 2 deletions blockinvgpfa/__init__.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
"""
:copyright: Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
:license: Modified BSD, see LICENSE.txt for details.
:license: Modified BSD, see LICENSE.txt for details.
"""
from .blockinvgpfa import BlockInvGPFA
from .preprocessing import EventTimesToCounts
Expand Down
4 changes: 1 addition & 3 deletions blockinvgpfa/blockinvgpfa.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
# ...
# Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
# Copyright 2014-2020 by the Elephant team.
# Modified BSD, see LICENSE.txt for details.
# Modified BSD, see LICENSE.txt for details.
# ...

from __future__ import division, print_function, unicode_literals
Expand Down
3 changes: 1 addition & 2 deletions blockinvgpfa/preprocessing.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# ...
# Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
# license Modified BSD, see LICENSE.txt for details.
# ...

Expand Down Expand Up @@ -53,7 +52,7 @@ class EventTimesToCounts(sklearn.base.TransformerMixin):
Examples
--------
>>> import numpy as np
>>> from gpfa import EventTimesToCounts
>>> from blockinvgpfa import EventTimesToCounts
>>> bin_size = 0.1 # [s]
>>> t_stop = 0.8 # [s]
>>> X = [
Expand Down
6 changes: 2 additions & 4 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@

import sys
import os
from datetime import date

# include root path to allow autodoc to find blockgpfa module
sys.path.insert(0, os.path.abspath('../'))
Expand All @@ -24,9 +23,8 @@

project = 'BlockInvGPFA Documentation'
authors = 'Brooks M. Musangu and Jan Drugowitsch'
copyright = "2021-{this_year}, {authors}".format(
this_year=date.today().year, authors=authors)
release = '0.1.0.'
copyright = f"2021, {authors}"
release = '0.1.0'
version = '0.1.0'

# -- General configuration ---------------------------------------------------
Expand Down
6 changes: 3 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,12 @@ name = "blockinversegpfa"
version = "0.1.0"
description = "A scikit-learn-compatible GPFA implementation using block matrix inversion."
authors = [
{name = "Brooks M. MUSANGU", email = "brooks_musangu@hms.harvard.edu"},
{name = "Jan DRUGOWITSCH", email = "jan_drugowitsch@hms.harvard.edu"}
{name = "Brooks M. Musangu", email = "brooks_musangu@hms.harvard.edu"},
{name = "Jan Drugowitsch", email = "jan_drugowitsch@hms.harvard.edu"}
]
license = {file = "LICENSE.txt"}
readme = "README.md"
requires-python = ">=3.9"
requires-python = ">=3.10,<3.14"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As far as I understand, this excludes the current Python version 3.14. Is there a reason for doing this? Does the code not run on Python 3.14?

If it does, then we should remove the upper bound, as we generally cannot predict which future versions the code will work with.

If it does not, and the problem is easily fixable, then let's do that instead.

dependencies = [
"numpy==2.2.1",
"scikit-learn==1.6.0",
Expand Down
1 change: 0 additions & 1 deletion test/test_blockinvgpfa.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# ...
# Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
# license Modified BSD, see LICENSE.txt for details.
# ...
"""
Expand Down
1 change: 0 additions & 1 deletion test/test_preprocessing.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# ...
# Copyright 2021 Brooks M. Musangu and Jan Drugowitsch.
# license Modified BSD, see LICENSE.txt for details.
# ...
"""
Expand Down
Loading