Skip to content

Add DeepStateModel #1253

Merged
merged 36 commits into from
Jun 23, 2023
Merged

Add DeepStateModel #1253

merged 36 commits into from
Jun 23, 2023

Conversation

ostreech1997
Copy link
Collaborator

@ostreech1997 ostreech1997 commented May 2, 2023

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Implementation of the Deep State Space Model

Closing issues

@github-actions
Copy link

github-actions bot commented May 2, 2023

@github-actions github-actions bot temporarily deployed to pull request May 2, 2023 15:20 Inactive
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2023

Codecov Report

Merging #1253 (f063c6d) into master (b1ec0f5) will increase coverage by 0.64%.
The diff coverage is 95.71%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master    #1253      +/-   ##
==========================================
+ Coverage   88.11%   88.75%   +0.64%     
==========================================
  Files         178      193      +15     
  Lines       11170    11738     +568     
==========================================
+ Hits         9842    10418     +576     
+ Misses       1328     1320       -8     
Impacted Files Coverage Δ
etna/models/nn/deepstate/state_space_model.py 88.54% <88.54%> (ø)
etna/models/nn/__init__.py 100.00% <100.00%> (ø)
etna/models/nn/deepstate/__init__.py 100.00% <100.00%> (ø)
etna/models/nn/deepstate/deepstate.py 100.00% <100.00%> (ø)
etna/models/nn/deepstate/linear_dynamic_system.py 100.00% <100.00%> (ø)

... and 17 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions github-actions bot temporarily deployed to pull request May 2, 2023 15:28 Inactive
Copy link
Contributor

@martins0n martins0n left a comment

Choose a reason for hiding this comment

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

Can we add examples to neural network notebook?
And we should add some information about choosing right family of SSM

etna/models/nn/deepstate/deepstate.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/linear_dynamic_system.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/state_space_model.py Show resolved Hide resolved
@martins0n
Copy link
Contributor

martins0n commented Jun 6, 2023

And yet another thing - look at test_inference folder - it would be great to add test cases for deepstate there

@martins0n martins0n mentioned this pull request Jun 7, 2023
9 tasks
@ostreech1997 ostreech1997 self-assigned this Jun 7, 2023
etna/models/nn/deepstate/state_space_model.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/state_space_model.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/state_space_model.py Show resolved Hide resolved
etna/models/nn/deepstate/state_space_model.py Outdated Show resolved Hide resolved
self.RNN = nn.LSTM(
num_layers=self.num_layers, hidden_size=self.latent_dim, input_size=self.input_size, batch_first=True
)
self.projectors = nn.ModuleDict(
Copy link
Collaborator

Choose a reason for hiding this comment

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

In the paper there is also the third type of transformations for the bounded parameters -- may be we can create the separate issue to add this option + prediction intervals

etna/models/nn/deepstate/deepstate.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/deepstate.py Outdated Show resolved Hide resolved
etna/models/nn/deepstate/deepstate.py Outdated Show resolved Hide resolved
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

martins0n
martins0n previously approved these changes Jun 22, 2023
@alex-hse-repository alex-hse-repository dismissed stale reviews from martins0n and themself via 4268d19 June 23, 2023 05:20
@alex-hse-repository alex-hse-repository enabled auto-merge (squash) June 23, 2023 05:21
@github-actions github-actions bot temporarily deployed to pull request June 23, 2023 05:26 Inactive
@alex-hse-repository alex-hse-repository merged commit 3f4ce91 into master Jun 23, 2023
@Mr-Geekman Mr-Geekman mentioned this pull request Jun 30, 2023
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants