Skip to content

Conversation

@kohlerca
Copy link
Collaborator

@kohlerca kohlerca commented Oct 8, 2025

This PR is a revision of the existing Trials objects. Several improvements are implemented:

Functionality

  1. Added additional methods to retrieve spike train or analog signal data across the trial repetitions (get_spiketrains_trial_by_trial and get_analogsignals_trial_by_trial).
  2. Moved redundant code to the base class Trials, for improved maintenance.
  3. Refactored unit tests to include content checks. The tests are based on a base unit test class that is shared across different test cases and provides custom assertions to compare Neo objects.
  4. Changed name convention of arguments to identify trials as indexes (trial_index), instead of the previous trial_id or trial_number.

Documentation

  1. Module docstring text improved.
  2. General revision of all classes and methods docstrings, to harmonize style and add detailed content of the API.
  3. Added usage examples to TrialsFromBlock and TrialsFromList.

TODOs

  • Sphinx configuration to handle links to Neo classes in the documentation.

Cristiano Köhler added 30 commits June 18, 2025 17:42
…s.py to trials.py. GPFA methods and unit tests adjusted.
…s are accessed by indexes (instead of ambiguous terms ID or number).

Added deprecations with warning decorator and corresponding unit tests.
@coveralls
Copy link
Collaborator

coveralls commented Oct 8, 2025

Coverage Status

coverage: 88.323% (+0.06%) from 88.262%
when pulling 6f018b8 on INM-6:enh/trial_by_trial_methods
into 5392583 on NeuralEnsemble:master.

@kohlerca kohlerca requested a review from mdenker October 9, 2025 12:01
@kohlerca kohlerca added enhancement Editing an existing module, improving something documentation Indicates a need for improvements or additions to documentation maintenance labels Oct 9, 2025
@kohlerca kohlerca added this to the v1.2.0 milestone Dec 1, 2025
@kohlerca kohlerca marked this pull request as ready for review December 2, 2025 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Indicates a need for improvements or additions to documentation enhancement Editing an existing module, improving something maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants