Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Run file update #9

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

GitHubEmploy
Copy link
Contributor

run file to make it more efficient.

Copy link
Owner

@tamuseanmiller tamuseanmiller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like the runfiles, just remove the unnecessary changes to the other files and I'll merge this in.

Copy link
Contributor Author

@GitHubEmploy GitHubEmploy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, all changes have been done

@tamuseanmiller
Copy link
Owner

I think it might just be worth it to drop train.py from the commit rather than me telling to to revert things, otherwise looks good.

--debug Specifies whether to use verbose logs during eval operation.
--stock-name=<stock-name> Specifies the specific stock to train
"""

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure how I didn't notice this before but why did you remove the helper comment?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants