Skip to content
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

Export torch models in torch module directly. #2685

Merged
merged 4 commits into from
Feb 23, 2023

Conversation

jaheba
Copy link
Contributor

@jaheba jaheba commented Feb 23, 2023

Allow to imports model more easily, e.g.:

from gluonts.torch import DeepAREstimator

# vs

from gluonts.torch.model.deepar import DeepAREstimator

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

@jaheba jaheba changed the title Export torch models in torch module directly. Export torch models in torch module directly. Feb 23, 2023
@lostella lostella added the enhancement New feature or request label Feb 23, 2023
Copy link
Contributor

@lostella lostella left a comment

Choose a reason for hiding this comment

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

🥳

@jaheba jaheba merged commit 1e35045 into awslabs:dev Feb 23, 2023
@jaheba jaheba deleted the torch-model-imports branch February 23, 2023 14:21
__path__ = extend_path(__path__, __name__) # type: ignore
from .model.deep_npts import DeepNPTSEstimator
from .model.deepar import DeepAREstimator
from .model.mqf2 import MQF2MultiHorizonEstimator
Copy link
Contributor

Choose a reason for hiding this comment

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

This line adds a dependency on cpflows which I’m not sure we want to require at this level. See https://github.com/awslabs/gluonts/blob/dev/requirements/requirements-extras-cpflows.txt

@lostella lostella added the torch This concerns the PyTorch side of GluonTS label Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request torch This concerns the PyTorch side of GluonTS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants