Skip to content

Suggestions PR #44

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 65 commits into
base: master
Choose a base branch
from
Draft

Suggestions PR #44

wants to merge 65 commits into from

Conversation

DanielAtKrypton
Copy link

Continuation of #6.

Daniel Kaminski de Souza added 30 commits May 3, 2020 02:47
🎨 Lint, format, remove not used imports, sort imports.
8d87d82
🐛 Fix search.py import bug.
1d9f542
👕 Clear training.ipynb output.
4ece89a
👕 lint make_npz.py
71c9edb
👕 lint datasets/README.md
bce27ba
👕 lint test/multiHeadAttention.py
f1fd606
🚧 Update labels.json to reflect the challenge.
8ab9035
🚧 Add model.py.
de9274c
🚧 Add dataset2.py.
6c35365
🚧 Improve pull request accordingly to feedback.
b1fb712
👕 dopout should actually be dropout.
c780755
Restore training.ipynb
2a8ba6d
➖ Remove csv2npz and make_npz.py
de92c1a
➕ Add pylint to requirements.txt.
5a7fd4e
👕 lint src/utils/search.py
6359785
👕 lint src/utils/search.py
6e86025
🚧 Add to ignore list *.csv
6de0f16
🚧 Development path
2fe0b18
🚧 Development path.
d4fe40c
🚧 Development path.
5285fd4
🎨 Resolution of conflicts.
0af294d
👕 lint code.
31e8d4a
🚀 Speed up transformer pytest.
906d883
🚀 Speed up pytest with seaborn passsengers dataset.
3304457
🐛 Fix FlightsDataset shape convention bug.
d7c1be3
➕ Add FlightsDataset labels.
d1019b3
➕ Add FlightsDataset labels.
ad80d93
🎨 Improve flights_dataset.
1384707
🎨 Start using the specialized MinMaxScaler object to scale the dataset.
1329e82
➕ Implement forecast functionality.
ad0479c
@DanielAtKrypton DanielAtKrypton mentioned this pull request Jan 5, 2021
Closed
@DanielAtKrypton DanielAtKrypton marked this pull request as draft January 5, 2021 15:04
@DanielAtKrypton
Copy link
Author

DanielAtKrypton commented Jan 5, 2021 •
edited
Loading

Relevant PR changes are:

  • d_model, output_model and input_model are back as originally conceived.
  • The time_series_predictor was updated in order to detect the Transformer model.
  • Numpy is not a dependency anymore because all operations could be made directly by Pytorch.
  • package name reverted to tst as originally conceived.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@DanielAtKrypton