All in one

[ramkdlr]

Hi gone through your EOD2 wonderful initiative could you merge the delivery and EOD OHLCV to form a single CSV?.

[BennyThadikaran]

@ramkdlr
Hi, I'm glad you bought this up. 👍

It has many benefits:

• It will reduce the number of files and save repo space.
• Some overall code reduction
• Users may find it easier to work with one file

I had considered this in the past but dismissed it. My concern was whether NSE would discontinue or change the structure of its delivery reports. It is not a concern anymore.

I'll start working on EOD2 next month and will create a github issue when i start work on this.
Meanwhile, I look into this and post my thoughts here.

[ramkdlr]

Great will wait for the update.

[BennyThadikaran]

I just had a quick look at this today. Below is how the columns would look after the merge.

Open, High, Low, Close, Volume, TOTAL_TRADES, QTY_PER_TRADE, DLV_QTY

TTL_TRD_QNTY is same as Volume.
NO_OF_TRADES renamed to TOTAL_TRADES
DELIV_QTY renamed to DLV_QTY

I'll start work on this next week.

[ramkdlr]

ok great.

[BennyThadikaran]

@ramkdlr Raised issue #62 . Work is almost done. But need to make changes in my other scripts and some final testing before i merge. So most likely by tomorrow evening.

[BennyThadikaran]

@ramkdlr I have completed this feature. See all the changes #69

I will post a Changelog and Update instructions on Discussions tomorrow.