HyukjinKwon opened a new pull request #32649:
URL: https://github.com/apache/spark/pull/32649


   ### What changes were proposed in this pull request?
   
   This PR enables plot tests with plotly
   
   ```bash
   ./python/run-tests --python-executables=python3 --modules=pyspark-pandas
   ```
   
   **Before**:
   
   ```
   Traceback (most recent call last):
     File "/.../miniconda3/envs/python3.8/lib/python3.8/runpy.py", line 194, in 
_run_module_as_main
       return _run_code(code, main_globals, None,
     File "/.../miniconda3/envs/python3.8/lib/python3.8/runpy.py", line 87, in 
_run_code
       exec(code, run_globals)
     File "/.../pyspark/pandas/tests/plot/test_frame_plot_plotly.py", line 42, 
in <module>
       plotly_requirement_message + " Or pandas<1.0; pandas<1.0 does not 
support latest plotly "
   TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'
   
   ```
   
   **After**:
   
   ```
   Starting test(python3): pyspark.pandas.tests.plot.test_series_plot_plotly
   ...
   Finished test(python3): pyspark.pandas.tests.plot.test_series_plot_plotly 
(23s)
   ...
   Finished test(python3): 
pyspark.pandas.tests.plot.test_series_plot_matplotlib (70s)
   ...
   Finished test(python3): pyspark.pandas.strings (60s)
   Finished test(python3): pyspark.pandas.series (135s)
   Finished test(python3): pyspark.pandas.window (43s)
   Finished test(python3): pyspark.pandas.frame (335s)
   Tests passed in 1296 seconds
   ```
   
   ### Why are the changes needed?
   
   For test coverage.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, test-only.
   
   ### How was this patch tested?
   
   By running the tests.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to