Lstm 50
Description
This merges the lstm_50 branch, which has been running to produce the test site, into main, so that the beta and prod versions of the website data pipeline will pull and process output from the lstm < 50 model and push it to s3, thereby updating the beta and prod websites to display the output from the lstm < 50 model instead of the lstm < 30 model.
Changes Made
Pull down model output for the lstm <50 model instead of the lstm <30 model. No processing changes were needed other than to update the naming convention for the model in the final formatted output parquet file.
How to Test
Already tested through deployment of the data pipeline to build the test tier of the website. Confirm that changes look good and that there are no security concerns with these changes.
MR t-shirt size
- extra-small (e.g., update color scheme)
- small (e.g., add tooltip to chart)
- medium (e.g., add new static chart, add new pipeline output)
- large (e.g., initial data processing pipeline, interactive chart)
Timeline for review
- ASAP - blocker for deployment
- Today - blocker for ongoing work
- Next couple of days
- In the next week
- By XX/XX/XX
- When reviewer has availability
Review Needs
Does this MR include data processing, modeling, or visualization code that will require domain review prior to release?
-
Yes, and I have opened an issue to document the need for review, using the
DomainReviewissue template - Yes, and a domain review issue already exists
- No
Related Issues
NA
Additional Notes
NA
Merge Request Checklists
-
Code changes adhere to best practices documented in
README.md -
Code has been cleaned the way Vue likes it - run
npm run lint - If applicable, the need for future domain review has been documented in an issue
-
Below section documents which browsers the site has been tested on:
- Desktop/laptop
- Chrome
- Safari
- Edge
- Firefox
- Mobile device
- Chrome
- Safari
- Edge
- Firefox
- Desktop/laptop