Skip to content

Comments

Update csvdir.py#2

Open
peterfabakker wants to merge 3 commits intobartosh:masterfrom
peterfabakker:patch-1
Open

Update csvdir.py#2
peterfabakker wants to merge 3 commits intobartosh:masterfrom
peterfabakker:patch-1

Conversation

@peterfabakker
Copy link

Hi, I adjusted the CSVdir so it can handle different exchanges (as needed for live trading some tickets like GLD, GDX, and to get live data from VIX index on CBOE. Everything works the same if one doesnt put in the exchange in the filename, or the asset type, but if you do the asset get loaded that way and the calendar gets matched to NYSE. Later we might extend this to have other calendar mappings, but for now I map everything to NYSE.

Hi, I adjusted the CSVdir so it can handle different exchanges (as needed for live trading some tickets like GLD, GDX, and to get live data from VIX index on CBOE. Everything works the same if one doesnt put in the exchange in the filename, or the asset type, but if you do the asset get loaded that way and the calendar gets matched to NYSE. Later we might extend this to have other calendar mappings, but for now I map everything to NYSE.
@bartosh
Copy link
Owner

bartosh commented Jan 20, 2018

Hi,

I'm sorry, I missed this PR somehow. Can you send it to upstream zipine please? csvdir bundle is already merged into upstream master.

Thanks,
Ed

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