Skip to content

Move pyconfig&pyconfig_deprecated to maxtext/configs#3266

Merged
copybara-service[bot] merged 1 commit intomainfrom
hengtaoguo-pyconfig
Feb 27, 2026
Merged

Move pyconfig&pyconfig_deprecated to maxtext/configs#3266
copybara-service[bot] merged 1 commit intomainfrom
hengtaoguo-pyconfig

Conversation

@hengtaoguo
Copy link
Copy Markdown
Collaborator

@hengtaoguo hengtaoguo commented Feb 26, 2026

Description

Move pyconfig.py and pyconfig_deprecated.py to under src/maxtext/configs.

Tests

CI

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • I have performed a self-review of my code. For an optional AI review, add the gemini-review label.
  • I have necessary comments in my code, particularly in hard-to-understand areas.
  • I have run end-to-end tests tests and provided workload links above if applicable.
  • I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in our documentation.

@hengtaoguo hengtaoguo force-pushed the hengtaoguo-pyconfig branch 2 times, most recently from 47d46d5 to 4bb4045 Compare February 26, 2026 22:39
@hengtaoguo hengtaoguo changed the title Move pyconfig/pyconfig_deprecated Move pyconfig/pyconfig_deprecated to maxtext/configs Feb 26, 2026
@hengtaoguo hengtaoguo changed the title Move pyconfig/pyconfig_deprecated to maxtext/configs Move pyconfig&pyconfig_deprecated to maxtext/configs Feb 26, 2026
@hengtaoguo hengtaoguo marked this pull request as ready for review February 26, 2026 22:52
Copy link
Copy Markdown
Collaborator

@bvandermoon bvandermoon left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @hengtaoguo

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 27, 2026

Codecov Report

❌ Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/MaxText/maxengine_server.py 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

hengtaoguo added a commit that referenced this pull request Feb 27, 2026
Imported from GitHub PR #3266

# Description

Move `pyconfig.py` and `pyconfig_deprecated.py` to under `src/maxtext/configs`.

# Tests

CI

# Checklist

Before submitting this PR, please make sure (put X in square brackets):
- [x] I have performed a self-review of my code. For an optional AI review, add the `gemini-review` label.
- [x] I have necessary comments in my code, particularly in hard-to-understand areas.
- [x] I have run end-to-end tests tests and provided workload links above if applicable.
- [x] I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in [our documentation](https://maxtext.readthedocs.io/en/latest/development.html#adding-new-documentation-files).

Copybara import of the project:

--
fd028e0 by hengtaoguo <hengtaoguo@google.com>:

update

Merging this change closes #3266

FUTURE_COPYBARA_INTEGRATE_REVIEW=#3266 from AI-Hypercomputer:hengtaoguo-pyconfig fd028e0
PiperOrigin-RevId: 875963255
hengtaoguo added a commit that referenced this pull request Feb 27, 2026
Imported from GitHub PR #3266

# Description

Move `pyconfig.py` and `pyconfig_deprecated.py` to under `src/maxtext/configs`.

# Tests

CI

# Checklist

Before submitting this PR, please make sure (put X in square brackets):
- [x] I have performed a self-review of my code. For an optional AI review, add the `gemini-review` label.
- [x] I have necessary comments in my code, particularly in hard-to-understand areas.
- [x] I have run end-to-end tests tests and provided workload links above if applicable.
- [x] I have made or will make corresponding changes to the doc if needed, including adding new documentation pages to the relevant Table of Contents (toctree directive) as explained in [our documentation](https://maxtext.readthedocs.io/en/latest/development.html#adding-new-documentation-files).

Copybara import of the project:

--
fd028e0 by hengtaoguo <hengtaoguo@google.com>:

update

Merging this change closes #3266

FUTURE_COPYBARA_INTEGRATE_REVIEW=#3266 from AI-Hypercomputer:hengtaoguo-pyconfig fd028e0
PiperOrigin-RevId: 875963255
@copybara-service copybara-service Bot merged commit fcaecd2 into main Feb 27, 2026
41 of 42 checks passed
@copybara-service copybara-service Bot deleted the hengtaoguo-pyconfig branch February 27, 2026 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants