Skip to content

Commit 37dca66

Browse files
authored
Merge pull request #2 from GreenArt11/moex-data-merger
- Added function read_all_boards() that provides data from all of the boards (as it is in v.0.9.0). Default behavoiur changed to returning data from the primary board only. - Bug fixed
2 parents c41c6d3 + 78cc196 commit 37dca66

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

pandas_datareader/moex.py

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,8 @@ def _get_metadata(self):
133133
) # market and engine
134134

135135
if fields[14] == "1": # main board for symbol
136-
symbol = symbol.upper()
137-
boards[symbol] = fields[1]
136+
symbol_U = symbol.upper()
137+
boards[symbol_U] = fields[1]
138138

139139
if symbol not in markets_n_engines:
140140
raise IOError(
@@ -149,8 +149,8 @@ def _get_metadata(self):
149149
markets_n_engines[symbol] = list(set(markets_n_engines[symbol]))
150150
return markets_n_engines, boards
151151

152-
def read(self):
153-
"""Read data"""
152+
def read_all_boards(self):
153+
"""Read data from every board"""
154154

155155
markets_n_engines, boards = self._get_metadata()
156156
try:
@@ -208,7 +208,12 @@ def read(self):
208208
b = concat(dfs, axis=0, join="outer", sort=True)
209209
else:
210210
b = dfs[0]
211+
return b
211212

213+
def read(self):
214+
"""Read data from primary board for each ticker"""
215+
markets_n_engines, boards = self._get_metadata()
216+
b = self.read_all_boards()
212217
result = pd.DataFrame()
213218
for secid in list(set(b["SECID"].tolist())):
214219
part = b[b["BOARDID"] == boards[secid]]

0 commit comments

Comments
 (0)