Skip to content
This repository has been archived by the owner on Jun 10, 2023. It is now read-only.

Rl dev #39

Open
wants to merge 189 commits into
base: main
Choose a base branch
from
Open

Rl dev #39

wants to merge 189 commits into from

Conversation

Lam-Richard
Copy link

No description provided.

rewong03 and others added 30 commits October 20, 2022 12:13
.total_assets() calculates the total amount of assets in a portfolio at self.current_time

.update_testing_data updates data for .visualize() which are self.total_daily_assets and self.dates.
Given a time range of time, .update_testing_data would iterate through the range of time to update the two arrays (total_daily_assets and dates) and plot the two arrays against each other to visualize the change in asset amount of a portfolio
…o packaging

Trying to fix merge conflicts
prep for strategy.visualize()
Rewrote documentation to include more details. Added more comments in methods describing logic in more detail.

Reformatted code for better readability.
@Lam-Richard Lam-Richard requested a review from rewong03 February 7, 2023 06:59
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.