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

[ENH] faster test parameter settings for StatsForecastAutoARIMA #2527

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fkiraly
Copy link
Collaborator

@fkiraly fkiraly commented Apr 21, 2022

Faster test parameter settings for StatsForecastAutoARIMA in order to speed up the tests.
(lower orders for parameter search range)

@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label Apr 21, 2022
aiwalter
aiwalter previously approved these changes Apr 21, 2022
@fkiraly
Copy link
Collaborator Author

fkiraly commented Apr 21, 2022

ok, the truncate parameter seems to break it? 😢

@FedericoGarza, would you be able to have a quick look and come up with a test parameter setting that has good coverage but is fast to run? One fit with the current settings and the test data used everywhere takes 20 sec in the test suite, ranking among the slowest tests.

@AzulGarza
Copy link
Contributor

Yes, sure :)

@fkiraly
Copy link
Collaborator Author

fkiraly commented Jun 12, 2022

any help, @FedericoGarza?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Continuous integration, unit testing & package distribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants