new: _LoaderBase.collect()

[lisphilar]

Related issues

#851 and #853

What was changed

Close #853.
.collect() method was added to _LoaderBase class, which is the parent class of DataLoader. _LoaderBase will be also used as the parent class of LocalDataLoader class, which will be implemented for #851.

[sourcery-ai]

Sourcery Code Quality Report

✅  Merging this PR will increase code quality in the affected files by 3.82%.

Quality metrics	Before	After	Change
Complexity	1.99 ⭐	1.55 ⭐	-0.44 👍
Method Length	38.12 ⭐	39.15 ⭐	1.03 👎
Working memory	6.43 🙂	5.89 ⭐	-0.54 👍
Quality	70.60% 🙂	74.42% 🙂	3.82% 👍

Other metrics	Before	After	Change
Lines	191	231	40

Changed files	Quality Before	Quality After	Quality Change
covsirphy/loading/loaderbase.py	100.00% ⭐	99.80% ⭐	-0.20% 👎
example/scenario_analysis.py	54.12% 🙂	54.34% 🙂	0.22% 👍
tests/test_loading.py	82.38% ⭐	84.34% ⭐	1.96% 👍

Here are some functions in these files that still need a tune-up:

File	Function	Complexity	Length	Working Memory	Quality	Recommendation
example/scenario_analysis.py	main	4 ⭐	376 ⛔	9 🙂	49.48% 😞	Try splitting into smaller methods
tests/test_loading.py	TestDataLoader.test_dataloader	0 ⭐	83 🙂	10 😞	71.41% 🙂	Extract out complex expressions

Legend and Explanation

The emojis denote the absolute quality of the code:

• ⭐ excellent
• 🙂 good
• 😞 poor
• ⛔ very poor

The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request.

---

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Help us improve this quality report!

[codeclimate]

Code Climate has analyzed commit 83189d8 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 94.3% (0.2% change).

View more on Code Climate.