docs: bring back hierarchy in ParallelLoader docs#706
Conversation
There was a problem hiding this comment.
Pull request overview
Restores documentation clarity for ParallelLoader by reintroducing a list of supported data-loader inputs, aiming to address issue #482 about missing “hierarchy” information.
Changes:
- Added a “Hierarchy of Data Loaders” section to the
ParallelLoaderclass docstring. - Listed specific CSV loader classes and base classes (
PyTorchData,KaggleData) as accepted inputs.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ad-claw000
left a comment
ad-claw000
left a comment
There was a problem hiding this comment.
Applied autopep8 formatting to the new log messages to fix the pre-commit CI failure.
ad-claw000
left a comment
ad-claw000
left a comment
There was a problem hiding this comment.
Fixed the pre-commit CI failure by applying autopep8 formatting to the new log messages. Ready for review!
ad-claw000
left a comment
ad-claw000
left a comment
There was a problem hiding this comment.
Addressed the feedback: updated the docstring to clarify that ParallelLoader accepts any Subscriptable that yields (commands, blobs) and the dataset is passed to ingest().
Summary
ParallelLoaderdocstring to make it clear which classes it accepts as inputs.Verification
aperturedb/ParallelLoader.py.Fixes #482