search

rohan472000 / reliance-stock-price-prediction-LSTM

Implemented LSTM model to predict Reliance stock prices, achieving accurate forecasts for 10 days.
MIT License
9 stars 4 forks source link

Update Github Action #10

Closed joelmathewphilip closed 1 year ago

joelmathewphilip commented 1 year ago

Add trigger to trigger the workflow manually for testing

rohan472000 commented 1 year ago

its failing, kindly check.

joelmathewphilip commented 1 year ago

Please review and approve

rohan472000 commented 1 year ago

Really, you just removed PR branch on main from yml, so it worked otherwise it will still fail.

Take your time and fix it.

joelmathewphilip commented 1 year ago

I removed the PR trigger because the workflow tries to merge changes to the running branch. Since the workflow is running on your repo, it will not recognise my main branch and hence gives the error. Ideally this workflow should run only during pushes to main or a cron job.

rohan472000 commented 1 year ago

Ahh..let's try this then on main.