Snapchat / Snapchat/GiGL

Simplify use of term "Loading" for our APIs

Open
#163 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
85
Forks
19
Avg merge
4d 10h
Merged PRs (30d)
6

Description

We have a lot of types of `loaders` when using in-memory-SGS GiGL i.e. DataLoading TFLoader FileLoader NeighborLoader. We should try to make more specific naming here when we can, i.e. `TFDatasetOptions` [[1](https://github.com/Snapchat/GiGL/blob/main/python/gigl/common/data/dataloaders.py#L51)]-> `TFDataIngestionOptions ` and `_load_and_build_partitioned_dataset` [[2](https://github.com/Snapchat/GiGL/blob/main/python/gigl/distributed/dataset_factory.py)] to `_ingest_and_build_partitioned_dataset `

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.