Skip to content

LSTM tutorial using airline passenger data #38

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

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from
Open

Conversation

camckenn
Copy link

@camckenn camckenn commented Nov 29, 2022

Purpose

Explain the goal of this PR, if it addresses an existing issue be sure to link to it.

Addresses number #15. Complete tutorial which uses an LSTM (and get the hang of pre-commit hooks and all).

Tag each of the issues that you are addressing with this PR with "Closes #"

Closes #15

Describe the big picture of your changes here, perhaps using a bullet list if multiple changes are done to accomplish a single goal.
(If it accomplishes too many goals, it would be best to create separate PR's for each.)

  • Complete tutorial which uses an LSTM
  • Learn to deal with pre-commit hooks
  • Show progress in learning LSTMs

Type of change

What types of change is it?
Select the appropriate type(s) that describe this PR

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Repository chores (changes made to improve repository/workflow quality)
  • Documentation update
  • Other (please describe)

-Other

Example/tutorial

Checklist

Put an x in the boxes that apply.

  • My code functions as intended
  • I have used camelCase in my variable and function names
  • I have commented my code properly
  • I have provided the necessary prompts for automatic documentation
  • I have provided a toy case under the examples folder
  • I have addressed all the issues entirely

@camckenn camckenn self-assigned this Nov 29, 2022
@camckenn camckenn added the LSTM Everything related to LSTM label Nov 29, 2022
@dorukaks dorukaks changed the base branch from main to dev December 2, 2022 17:17
@dorukaks
Copy link
Contributor

dorukaks commented Dec 2, 2022

Changed the base branch from main to dev. After merging #35, we will freeze main and branch off from dev. Mentioned here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LSTM Everything related to LSTM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TODO] Initial LSTM Test from Tutorial
2 participants