Skip to content

Update trades.py#15

Open
mann-jdv wants to merge 1 commit intoossdaiict:mainfrom
mann-jdv:patch-2
Open

Update trades.py#15
mann-jdv wants to merge 1 commit intoossdaiict:mainfrom
mann-jdv:patch-2

Conversation

@mann-jdv
Copy link
Copy Markdown

Built the get_trades tool that fetches NSE Bhav Copy data for a date range, groups by stock symbol, and returns summarized metrics (total volume, total value, first opening price, last closing price) as a list of dictionaries.

main features:

Downloads data using get_data_for_date_range(), processes with pandas groupby operations, handles missing data/columns, and includes Windows encoding fix for console output.

Built the get_trades tool that fetches NSE Bhav Copy data for a date range, groups by stock symbol, and returns summarized metrics (total volume, total value, first opening price, last closing price) as a list of dictionaries.


main features:

 Downloads data using get_data_for_date_range(), processes with pandas groupby operations, handles missing data/columns, and includes Windows encoding fix for console output.
Copy link
Copy Markdown

@vaishcodescape vaishcodescape left a comment

Choose a reason for hiding this comment

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

Try to format the code a bit better

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants