Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

This change prepares the scratchgpt project to be a clean, usable package for PyPI. It updates the project metadata in pyproject.toml, defines a public API in scratchgpt/__init__.py, and ensures that all dependencies required for testing are included in the development environment.


PR created automatically by Jules for task 12966135966237720159

This commit prepares the `scratchgpt` project for packaging and distribution on PyPI.

Key changes include:
- Updated `pyproject.toml` with a project description, classifiers, and repository URLs to improve package metadata.
- Populated `scratchgpt/__init__.py` to define a clean public API, exposing the core `TransformerLanguageModel` and configuration classes.
- Added tokenizer dependencies to the `dev` group to ensure the test suite runs correctly in a development environment.
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll acknowledge your comments with a 👀 emoji and then get to work. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

I will automatically address your feedback. For any comments you don't want me to act on, just include (aside).



For security, I will only act on instructions from the user who triggered this task for this pull request.

@ayeganov ayeganov changed the base branch from main to feat/easy_to_use_interface September 12, 2025 16:30
Based on user feedback, this commit expands the public API of the `scratchgpt` package to make it more extensible and reusable.

Key changes include:
- Updated `scratchgpt/__init__.py` to export interfaces, classes, and functions from the `tokenizer`, `data`, `training`, and `model_io` modules.
- This allows users to import and build upon the library's core components, such as `DataSource`, `Trainer`, and `Tokenizer`.
@ayeganov ayeganov marked this pull request as ready for review September 12, 2025 16:39
@ayeganov ayeganov self-assigned this Sep 12, 2025
Base automatically changed from feat/easy_to_use_interface to main September 12, 2025 19:44
Copy link
Contributor

@ayeganov ayeganov left a comment

Choose a reason for hiding this comment

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

Good job Jules!

@ayeganov ayeganov merged commit dcf53c4 into main Sep 12, 2025
3 checks passed
@ayeganov ayeganov deleted the feature/prepare-for-pypi branch September 12, 2025 19:47
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.

2 participants