search

cdump / investments

Analysis of Interactive Brokers reports for tax reporting in Russia
MIT License
111 stars 20 forks source link

Просьба добавить поддержку валют. В частности CAD #39

Closed President closed 3 years ago

President commented 3 years ago

Traceback (most recent call last): File "/Users/kostya/.pyenv/versions/3.9.1/lib/python3.9/runpy.py", line 197, in _run_module_as_main return _run_code(code, main_globals, None, File "/Users/kostya/.pyenv/versions/3.9.1/lib/python3.9/runpy.py", line 87, in _run_code exec(code, run_globals) File "/Users/kostya/.local/lib/python3.9/site-packages/investments/ibdds/main.py", line 3, in main() File "/Users/kostya/.local/lib/python3.9/site-packages/investments/ibdds/ibdds.py", line 86, in main parser_object = parse_reports(args.activity_report_filepath) File "/Users/kostya/.local/lib/python3.9/site-packages/investments/ibdds/ibdds.py", line 39, in parse_reports parser_object.parse_csv(activity_report_filepath=activity_report) File "/Users/kostya/.local/lib/python3.9/site-packages/investments/ibdds/ibdds.py", line 27, in parse_csv self._real_parse_activity_csv(csv.reader(activity_fh, delimiter=','), { File "/Users/kostya/.local/lib/python3.9/site-packages/investments/report_parsers/ib.py", line 211, in _real_parse_activity_csv parser_fn(fields) File "/Users/kostya/.local/lib/python3.9/site-packages/investments/report_parsers/ib.py", line 327, in _parse_cash_report currency = Currency.parse(currency_code) File "/Users/kostya/.local/lib/python3.9/site-packages/investments/currency.py", line 17, in parse raise ValueError(strval) ValueError: CAD

cdump commented 3 years ago

Только CAD есть в том отчете или другие валюты помимо USD?

Rom888 commented 3 years ago

Присоединяюсь к просьбе, у меня USD и EUR. Если нужен пример активити репорта брокера - пришлю куда скажете.

esemi commented 3 years ago

@President @Rom888 Коллеги, а можно примерчик отчёта с разными валютами? Пробую сделать эту фичу

Rom888 commented 3 years ago

@esemi на какую почту можно прислать пример отчета с разными валютами?

esemi commented 3 years ago

@esemi на какую почту можно прислать пример отчета с разными валютами?

@Rom888 support@esemi.ru Или можно удалить перс данные и выложить на диск куда?)

Rom888 commented 3 years ago

@esemi
отправил на почту, спасибо за внимание к этому issue.

esemi commented 3 years ago

Прогнал по живому отчёту тулу. Всё должно работать корректно только с одним ограничением - комиссия и налог по дивам/сделкам должны быть уплачены в валюте операции. Другими словами, если бумага продавалась за одну валюту, а комиссии за эту операцию списали в другой - тула такое не прожуёт и сообщит об этом.

Для дивидендов это не так критично и данную проверку можно убрать, но вот для сделок это сложнее, поскольку комиссии учитываются в цене бумаги при купле/продаже и сейчас разные валюты там не сложить.

Но в целом должно работать, мерж реквест есть, ждём @cdump чтобы проверить да влить в тулу)