Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create rule S6983 #3973

Merged
merged 4 commits into from
Sep 16, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions rules/S6983/metadata.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
{
}
25 changes: 25 additions & 0 deletions rules/S6983/python/metadata.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
{
"title": "The \"num_workers\" parameter should be specified for \"torch.utils.data.DataLoader\"",
"type": "CODE_SMELL",
"status": "ready",
"remediation": {
"func": "Constant\/Issue",
"constantCost": "2min"
},
"tags": [
"pytorch",
"machine-learning"
],
"defaultSeverity": "Minor",
"ruleSpecification": "RSPEC-6983",
"sqKey": "S6983",
"scope": "All",
"defaultQualityProfiles": ["Sonar way"],
"quickfix": "targeted",
"code": {
"impacts": {
"RELIABILITY": "LOW"
},
"attribute": "COMPLETE"
}
}
70 changes: 70 additions & 0 deletions rules/S6983/python/rule.adoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
This rule raises an issue when a `torch.utils.data.Dataloader` is instantiated without specifying the `num_workers` parameter.

== Why is this an issue?

In the PyTorch library, the data loaders are used to provide an interface where common operations such as batching can be implemented.
It is also possible to parallelize the data loading process by using multiple worker processes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not 100% sure here but I think worker should plurialized

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should not be pluralized

This can improve performance by increasing the number of batches being fetched in parallel, at the cost of higher memory usage.
This performance increase can also be attributed to avoiding the Global Interpreter Lock (GIL) in the Python interpreter.


== How to fix it
Specify the `num_workers` parameter when instantiating the `torch.utils.data.Dataloader` object.

The default value of `0` will use the main process to load the data, and might be faster for small datasets that can fit completely in memory.

For larger datasets, it is recommended to use a value of `1` or higher to parallelize the data loading process.

=== Code examples

==== Noncompliant code example

[source,python,diff-id=1,diff-type=noncompliant]
----
from torch.utils.data import DataLoader
from torchvision import datasets
from torchvision.transforms import ToTensor

train_dataset = datasets.MNIST(root='data', train=True, transform=ToTensor())
train_data_loader = DataLoader(train_dataset, batch_size=32)# Noncompliant: the num_workers parameter is not specified
----

==== Compliant solution

[source,python,diff-id=1,diff-type=compliant]
----
from torch.utils.data import DataLoader
from torchvision import datasets
from torchvision.transforms import ToTensor

train_dataset = datasets.MNIST(root='data', train=True, transform=ToTensor())
train_data_loader = DataLoader(train_dataset, batch_size=32, num_workers=4)
----

== Resources
=== Documentation

* PyTorch documentation - https://pytorch.org/docs/stable/data.html#single-and-multi-process-data-loading[Single- and Multi-process Data Loading]

* PyTorch documentation - https://pytorch.org/tutorials/beginner/basics/data_tutorial.html[Datasets and DataLoaders]

ifdef::env-github,rspecator-view[]

(visible only on this page)

== Implementation specification


=== Message

Primary : Specify the `num_workers` parameter.

=== Issue location

Primary : Name of the instantiation

=== Quickfix

Fill in with the default parameter

endif::env-github,rspecator-view[]
Loading