Skip to content

[monarch/ext] fold monarch_meta_extension into actor_extension #463

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

Open
wants to merge 2 commits into
base: gh/suo/52/base
Choose a base branch
from

Conversation

suo
Copy link
Contributor

@suo suo commented Jul 8, 2025

Stack from ghstack (oldest at bottom):

Right now we have 3 extensions:

  • actor_extension
  • monarch_extension (will be turned into tensor_engine_extension)
  • monarch_meta_extension (internal-only).

This diff folds monarch_meta_extension into actor_extension to reduce the number of independent shared objects, using condition compilation to make the OSS build work.

Differential Revision: D77941251

Right now we have 3 extensions:
- actor_extension
- monarch_extension (will be turned into tensor_engine_extension)
- monarch_meta_extension (internal-only).

This diff folds `monarch_meta_extension` into `actor_extension` to reduce the number of independent shared objects, using condition compilation to make the OSS build work.

Differential Revision: [D77941251](https://our.internmc.facebook.com/intern/diff/D77941251/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77941251

…sion"

Right now we have 3 extensions:
- actor_extension
- monarch_extension (will be turned into tensor_engine_extension)
- monarch_meta_extension (internal-only).

This diff folds `monarch_meta_extension` into `actor_extension` to reduce the number of independent shared objects, using condition compilation to make the OSS build work.

Differential Revision: [D77941251](https://our.internmc.facebook.com/intern/diff/D77941251/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77941251

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants