Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix tdx industry info parser error, refactor with a general logic. #1782

Merged
merged 1 commit into from
Jan 15, 2022

Conversation

oraix
Copy link

@oraix oraix commented Jan 15, 2022

QUANTAXIS PR 😇

[CHANGELOG]

  • 修复QA_fetch_get_tdx_industry 无法获得incon.dat文件问题和tdxhy.cfg字段变更(没有swhy了)导致的行业板块解析错误
  • 重构 QA_fetch_get_stock_block, 和QA_fetch_get_tdx_industry, 优先使用api获取incon.dat, 如果失败则从zhb.zip文件获取
  • QA_fetch_get_tdx_industry支持从通达信软件T0002\hq_cache目录中获取行业数据.

🛠该PR主要解决的问题🛠:

  • 修复报错通达信板块数据的问题:
    [Errno 2] No such file or directory: 'C:\\Users\\xxx\\AppData\\Local\\Temp\\tdx_857180/incon.dat' Error save_tdx.QA_SU_save_stock_block exception!

  • 相关issue: 关于保存板块数据失败的问题 #1742

作者信息:
时间: 2022-01-16

@pep8speaks
Copy link

Hello @oraix! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1311:65: E202 whitespace before ')'
Line 1311:80: E501 line too long (86 > 79 characters)
Line 1312:80: E501 line too long (86 > 79 characters)
Line 1313:80: E501 line too long (86 > 79 characters)
Line 1314:80: E501 line too long (86 > 79 characters)
Line 1315:65: E202 whitespace before ')'
Line 1315:80: E501 line too long (86 > 79 characters)
Line 1316:65: E202 whitespace before ')'
Line 1316:80: E501 line too long (86 > 79 characters)
Line 1317:80: E501 line too long (88 > 79 characters)
Line 1318:80: E501 line too long (88 > 79 characters)
Line 1319:80: E501 line too long (88 > 79 characters)
Line 1320:80: E501 line too long (88 > 79 characters)
Line 1322:80: E261 at least two spaces before inline comment
Line 1322:80: E501 line too long (117 > 79 characters)
Line 1327:32: E261 at least two spaces before inline comment
Line 1336:32: E201 whitespace after '('
Line 1336:80: E501 line too long (88 > 79 characters)
Line 1336:87: E202 whitespace before ')'
Line 1341:1: E302 expected 2 blank lines, found 1
Line 1341:41: E231 missing whitespace after ':'
Line 1346:23: E701 multiple statements on one line (colon)
Line 1351:40: E231 missing whitespace after ':'
Line 1351:80: E501 line too long (91 > 79 characters)
Line 1351:88: E202 whitespace before '}'
Line 1351:90: E202 whitespace before ')'
Line 1356:1: E302 expected 2 blank lines, found 1
Line 1356:31: E231 missing whitespace after ':'
Line 1364:65: E261 at least two spaces before inline comment
Line 1364:80: E501 line too long (110 > 79 characters)
Line 1390:1: E302 expected 2 blank lines, found 1
Line 1400:80: E501 line too long (100 > 79 characters)
Line 1405:1: E302 expected 2 blank lines, found 1
Line 1412:80: E501 line too long (90 > 79 characters)
Line 1414:80: E501 line too long (102 > 79 characters)

@yutiansut yutiansut merged commit 51e8120 into yutiansut:master Jan 15, 2022
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.

3 participants