BUG #21374: Fix divison with complex numbers in eval() method

[fillipe-gsm]

• closes pd.eval() discards imaginary part in division "/" #21374
• tests added / passed

data = {"a": [1 + 2j], "b": [1 + 1j]}
df = pd.DataFrame(data = data)
df.eval("a/b")

Expected output:

df["a"]/df["b"]

0    (1.5+0.5j)
dtype: complex128

New result:

df.eval("a/b")

0    (1.5+0.5j)
dtype: complex128

• passes git diff upstream/master -u -- "*.py" | flake8 --diff
• whatsnew entry

[pep8speaks]

Hello @fillipe-gsm! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on June 14, 2018 at 02:15 Hours UTC

[codecov]

Codecov Report

Merging #21472 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master   #21472      +/-   ##
==========================================
- Coverage    91.9%   91.89%   -0.01%     
==========================================
  Files         153      153              
  Lines       49606    49606              
==========================================
- Hits        45589    45587       -2     
- Misses       4017     4019       +2

Flag	Coverage Δ
#multiple	90.29% <100%> (-0.01%)	⬇️
#single	41.89% <0%> (ø)	⬆️

Impacted Files	Coverage Δ
pandas/core/computation/ops.py	95.6% <100%> (ø)	⬆️
pandas/util/testing.py	84.6% <0%> (-0.21%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5fc769...72252e1. Read the comment docs.

[jreback]

tests!

[jreback]

needs tests and a whatsnew entry (bug fix in numeric), can be in 0.23.2

[jbrockmendel]

@fillipe-gsm this is almost ready, just needs a unit test and what’s new entry.

[jreback]

closing. needs tests. pls ping if able to update.