EricPostMaster / fortune-cookie-movies

Web app that imagines how much cooler fortune cookies would be if the messages inside were movie plots. πŸŽ¬πŸ“½πŸΏπŸ₯ 
MIT License
11 stars 4 forks source link

Small changes. Refactored verb loop. i not needed #1

Closed xjseabrum closed 2 years ago

xjseabrum commented 2 years ago

Pretty small changes. I was able to look at the code and step through how the verb replacement was happening. The index i is factored out of the code as a result and recursion doesn't need to happen. Also added in a file called test_cases.py which contains the sentences that you had commented out at the bottom of the fortune_cookie.py file.

I wanted to see if I could take a crack at the adverbial phrase detection; however, still have to do my research responsibilities. Hope the changes are helpful Eric!

xjseabrum commented 2 years ago

Adding in the test_cases.py file from earlier.

xjseabrum commented 2 years ago

Wait, found an error, will submit another pull request soon.