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

Add ExtractPopularImages, WriteGEXF, and WriteGraphML to Python. #466

Draft
wants to merge 2 commits into
base: master
from

Conversation

@ruebot
Copy link
Member

ruebot commented May 25, 2020

GitHub issue(s): #409

What does this Pull Request do?

Add ExtractPopularImages, WriteGEXF, and WriteGraphML to Python.

  • Partially addresses #409
  • Clean up formatting in app.py
  • Cleanup doc comments on the Scala side
  • TODO: Extract Entities

How should this be tested?

ruebot added 2 commits May 25, 2020
- Partially addresses #409
- Clean up formatting in app.py
- Cleanup doc comments on the Scala side
- TODO: Extract Entities
from pyspark.sql.functions import desc, first


def Extract_Popular_Images(d: DataFrame, limit: int, min_width: int, min_height: int):

This comment has been minimized.

Copy link
@ruebot

ruebot May 25, 2020

Author Member

Oh, I don't need to do the type hinting here. I'll yank this before we merge in the end.

@ruebot ruebot requested review from ianmilligan1 and lintool May 25, 2020
@codecov
Copy link

codecov bot commented May 25, 2020

Codecov Report

Merging #466 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #466   +/-   ##
=======================================
  Coverage   75.86%   75.86%           
=======================================
  Files          49       49           
  Lines        1442     1442           
  Branches      279      279           
=======================================
  Hits         1094     1094           
  Misses        218      218           
  Partials      130      130           
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.