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

[timeseries] add some basic block and deepAR model #2027

Merged
merged 12 commits into from
Oct 28, 2022

Conversation

Carkham
Copy link
Contributor

@Carkham Carkham commented Sep 19, 2022

Description

This PR adds two new sections for creating trainable models.

The first part is embedding and scaler, which are responsible for encoding the input features and scaling the target values.

The second part is that we implemented a deepAR network model with the above-mentioned blcok, which refers to the implementation in gluonTS.

Since the model predicts a probability distribution, we need to override the different forward methods when training and inference with different block.

@Carkham Carkham marked this pull request as draft September 19, 2022 08:49
@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2022

Codecov Report

Base: 72.08% // Head: 71.36% // Decreases project coverage by -0.72% ⚠️

Coverage data is based on head (644150a) compared to base (bb5073f).
Patch coverage: 71.66% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2027      +/-   ##
============================================
- Coverage     72.08%   71.36%   -0.73%     
- Complexity     5126     6275    +1149     
============================================
  Files           473      624     +151     
  Lines         21970    27795    +5825     
  Branches       2351     2997     +646     
============================================
+ Hits          15838    19835    +3997     
- Misses         4925     6500    +1575     
- Partials       1207     1460     +253     
Impacted Files Coverage Δ
api/src/main/java/ai/djl/modality/cv/Image.java 69.23% <ø> (-4.11%) ⬇️
...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java 76.00% <ø> (ø)
...rc/main/java/ai/djl/modality/cv/output/Joints.java 71.42% <ø> (ø)
.../main/java/ai/djl/modality/cv/output/Landmark.java 100.00% <ø> (ø)
...main/java/ai/djl/modality/cv/output/Rectangle.java 72.41% <0.00%> (ø)
...i/djl/modality/cv/translator/BigGANTranslator.java 21.42% <0.00%> (-5.24%) ⬇️
.../modality/cv/translator/ImageFeatureExtractor.java 0.00% <0.00%> (ø)
.../ai/djl/modality/cv/translator/YoloTranslator.java 27.77% <0.00%> (+18.95%) ⬆️
...modality/cv/translator/wrapper/FileTranslator.java 44.44% <ø> (ø)
...y/cv/translator/wrapper/InputStreamTranslator.java 44.44% <ø> (ø)
... and 563 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Carkham Carkham marked this pull request as ready for review September 19, 2022 14:09
@KexinFeng KexinFeng changed the title [timeseries] add some basic block and deepar model [timeseries] add some basic block and deepAR model Sep 19, 2022
@KexinFeng KexinFeng force-pushed the dev-deepar branch 5 times, most recently from ae48241 to c873fc3 Compare October 26, 2022 07:50
* and limitations under the License.
*/

package ai.djl.timeseries.block;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add unit test

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it.

@frankfliu frankfliu merged commit fe4a9f3 into deepjavalibrary:master Oct 28, 2022
patins1 pushed a commit to patins1/djl that referenced this pull request Oct 30, 2022
)

* feature: add TimeSeriesDataset and training transform
* feature: some basic block and deepar model
* feature: add train example
* feature: add m5-demo and air passengers demo

Co-authored-by: Carkham <1302112560@qq.com>
Co-authored-by: Frank Liu <frankfliu2000@gmail.com>
Co-authored-by: KexinFeng <fengx463@umn.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants