-
Notifications
You must be signed in to change notification settings - Fork 4
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
Apibranch #112
Open
alasali1
wants to merge
56
commits into
backend
Choose a base branch
from
apibranch
base: backend
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fixed folder layout
React S3 Action deploy
* first * update on assignment * Modified task.py * deleted floop_data_15k.json and word2vec11.model * removed the commented * word2vec11.model file added * updated * update line 83 * update csv file * updates of * Update on the code * floop_data_15k.json deleted * DataFromS3 to Field1 changed * line 25 added * Update on task.py and added info.md file
* feat(makeReactBucket.sh): bypass user input with shell arg * feat(makeReactBucket.sh): make script callable from anywhere * chore(makeReactBucket.sh): make quotes convention more consistent * feat(makeReactBucket.sh): extract initials validation function * refactor(/devops): moved function to a helper file * refactor(/devops): make function take arg instead of use global * chore(/devops): lint, change from .sh to .bash * refactor(makeReactBucket.json): converted to yml for consistency with other template files * chore(makeReactBucket.json): delete file * Merge development into dj-40 commit 29f3ca6 Author: Daniel <[email protected]> Date: Thu Feb 3 16:02:49 2022 -0800 Python script run feedback datasets#22 (#30) * first * update on assignment * Modified task.py * deleted floop_data_15k.json and word2vec11.model * removed the commented * word2vec11.model file added * updated * update line 83 * update csv file * updates of * Update on the code * floop_data_15k.json deleted * DataFromS3 to Field1 changed * line 25 added * Update on task.py and added info.md file commit f293101 Merge: 4a8da9b 6f07a1e Author: ChrisNavo <[email protected]> Date: Thu Feb 3 14:11:15 2022 -0800 Merge pull request #59 from North-Seattle-College/react-S3-action React S3 Action deploy commit 6f07a1e Author: CNavo <[email protected]> Date: Wed Feb 2 21:50:37 2022 -0800 React S3 Action deploy
Co-authored-by: Dale Berg <[email protected]>
…riable (#63) * feat(makeReactBucket.bash): add region arg * style(makeReactBucket.bash): lint, add function comment * feat(makeReactBucket.bash): add env var * fix(makeReactBucket.bash): fix when called by . ./
…tion S3 deploy github action
uses fetch to send request determines if error using response.ok sets response or error data to be passed up
caught Boolean mistake
…nces Python Regex for Splitting Feedback into Sentences stored in a List
Create a Submit button that handles POST request
Feedback analysis
Created Error Message Component
Create a component for the feedback text
new PR for deploying lambda with GitHub Action
* first * update on assignment * Modified task.py * deleted floop_data_15k.json and word2vec11.model * removed the commented * word2vec11.model file added * updated * update line 83 * update csv file * updates of * Update on the code * floop_data_15k.json deleted * DataFromS3 to Field1 changed * line 25 added * Update on task.py and added info.md file * sendiment analysis
* Added notebook file that works in SageMaker * Rename sage-maker-clustering.ipynb to backend/sage-maker-clustering.ipynb
determin if sentence is a question
Updates script to upload contents of simpleLambda folder as a zip file
Create Automation for API Gateway
* chore(floop_data_collection.py): lint * feat(floop_data_collection.py): get each message in a convo * refactor(floop_data_collection.py): extract fn for getting data * refactor(floop_data_collection.py): make get_data recursive, enforce a convo limit * fix(floop_data_collection.py): pass dupe_list to get_conversations * build(/data): add requirements.txt, argparse * feat(floop_data_collection.py): ✨ add optional command line args, processed count * fix(floop_data_collection.py): 🐛 move print statement within loop * fix(floop_data_collection.py): ⚡ move dupe check above message collection call * style(floop_data_collection.py): 🚨 lint * chore(floop_data_collection.py): 🔥 remove console log * feat(floop_data_collection.py): ✨ scrub pii with scrubadub * feat(floop_data_collection.py): ✨ add env arg to set db path * style(floop_data_collection.py): 🚨 lint * refactor(floop_data_collection.py): 🎨 replace strings with constants * refactor(floop_data_collection.py): 🎨 remove unnecessary variable * feat(floop_data_collection.py): ✨ add entity detector
Styling added to UI
…sation (#104) Co-authored-by: Dale Berg <[email protected]>
S3 action updates
Updatethis API gateway was tested on an older version of the lambda function that was not functional. The lambda function has now been updated and deployed. DescriptionAs a user, I want to post a feedback message through the API gateway. GalleryTesting in Postman Tested
TestingPrerequisites
Testing in Postman
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please read update in comments below
Swagger specification with host as lambda function, along with the JSON version uploaded to AWS API gateway.
issue #43
TESTING

To Test the API in AWS. Log in to AWS, look for simplePostLambda under lambda functions. Under the triggers look for the API gateway called AI Feedback Coach
Click on the lightning shaped button under test

At the bottom there is a textbox for the body. In this text box type the message and id for the feedback resource as shown below

As you will see we get a 500 Internal Server error. That is not supposed to happen. The yaml document originally works with sites such as http bin but I have yet to successfully integrate it to our Lambda function.