Skip to content

Add DataLoader implementation - #7

Merged
Enskc05 merged 2 commits into
mainfrom
ml/feature/dataloader
Jul 16, 2026
Merged

Enskc05 merged 2 commits into
mainfrom
ml/feature/dataloader

Conversation

@edatosun

@edatosun edatosun commented Jul 16, 2026 •

Copy link
Copy Markdown
Collaborator
  • Added Dataset implementation
  • Added DataLoader with batch support
  • Prepared data loading pipeline for model training

Copilot AI review requested due to automatic review settings July 16, 2026 13:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces the initial PyTorch dataset + dataloader building blocks under services/ml/data/, intended to support the ML training data pipeline (loading paired clean/ and degraded/ .npy samples for training).

Changes:

  • Added BlackHoleDataset for loading paired clean/degraded .npy samples as tensors.
  • Added create_dataloader helper to construct a PyTorch DataLoader for the dataset.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
services/ml/data/dataset.py Adds a PyTorch Dataset for paired clean/degraded .npy training samples.
services/ml/data/dataloader.py Adds a create_dataloader helper to build a PyTorch DataLoader over the dataset.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +1 to +5
import numpy as np
import torch
from torch.utils.data import Dataset
from pathlib import Path

Comment on lines +1 to +5
from torch.utils.data import DataLoader
from .dataset import BlackHoleDataset


def create_dataloader(root_dir, batch_size=16):
@Enskc05
Enskc05 merged commit 3e49916 into main Jul 16, 2026
1 check passed
@Enskc05
Enskc05 deleted the ml/feature/dataloader branch July 16, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants