SainsburyWellcomeCentre / aeon_mecha

Project Aeon's main library for interfacing with acquired data. Contains modules for raw data file io, data querying, data processing, data qc, database ingestion, and building computational data pipelines.
BSD 3-Clause "New" or "Revised" License
5 stars 6 forks source link

fix(Pose): remove config_file from class property #384

Closed ttngu207 closed 2 months ago

ttngu207 commented 2 months ago

Summary by CodeRabbit

github-actions[bot] commented 2 months ago

Image description CodeRabbit

Walkthrough

The proposed changes primarily focus on the handling of configuration files within the Reader class in the aeon/io/reader.py file. The config_file class property has been removed and instead, it is passed as a parameter to several methods. This modification enhances modularity and maintainability by reducing the reliance on shared state.

Changes

File Summary
aeon/io/reader.py Removed the config_file class property and passed it as a parameter to the get_class_names, get_bodyparts, and class_int2str methods. This change improves modularity and maintainability but may impact the external interface or behavior of the code.

Uplevel your code reviews with CodeRabbit Pro ### CodeRabbit Pro If you like this project, please support us by purchasing the [Pro version](https://coderabbit.ai). The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.
jkbhagatio commented 2 months ago

Actually this is good as is, so will merge like this