Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to the
pyprobe
package, primarily focusing on enhancing type annotations and simplifying class structures. The most important changes involve updating theinfo
field to support nested dictionaries and removing re-definitions of inherited attributes from multiple classes.Type Annotations and Field Updates:
pyprobe/cell.py
: Updated theinfo
field in theCell
class to support nested dictionaries. This change is reflected in the type annotations and the corresponding field validator. [1] [2] [3]Codebase Simplification:
pyprobe/filters.py
: Removed inherited attributes such asbase_dataframe
,info
, andcolumn_definitions
from multiple classes (Procedure
,Experiment
,Cycle
,Step
). This helps in simplifying the class definitions and reducing clutter. [1] [2] [3] [4]pyprobe/rawdata.py
: Removed thebase_dataframe
andinfo
attributes from theRawData
class and added a detailed docstring for thestep_descriptions
attribute.Additional Changes:
pyprobe/result.py
: Updated theinfo
field in theResult
class to support nested dictionaries and adjusted the type annotations accordingly. [1] [2]pyprobe/filters.py
: Removed the import ofField
frompydantic
as it is no longer needed.