FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

README and `.gitignore` update to add GraphFrames classes by rjurney · Pull Request #629 · graphframes/graphframes · GitHub

/ graphframes Public

README and .gitignore update to add GraphFrames classes - #629

Merged
SemyonSinchenko merged 64 commits into
masterfrom
rjurney/readme-3
Jul 14, 2025
ConversationCommits (64)ChecksFiles changed
Merged

README and .gitignore update to add GraphFrames classes#629
SemyonSinchenko merged 64 commits into
masterfrom
rjurney/readme-3

Conversation

rjurney commented Jul 12, 2025

Copy link
Copy Markdown
Collaborator

What changes were proposed in this pull request?

Why are the changes needed?

rjurney added 30 commits February 15, 2025 20:58
Converted tests to pytest. Build a Python package. Update requirement…
f4e9cdb 
…s.txt and split out requirements-dev.txt. Version bumps.
Restore Python .gitignore
c256244
Extra newline removed
6c3df0b
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
b2838d2 
…ney/build-upgrades
Added VERSION file set to 0.8.5
caf5091
isort; fiex edgesDF variable name.
7cfa2d1
Merge branch 'master' of github.com:graphframes/graphframes into rjur…
2ca9a15 
…ney/build-upgrades
Back out Dockerfile changes
a8bf0be
Back out version change in build.sbt
54a942d
Backout changes to config and run-tests
8b0e346
Back out pytest conversion
46c2b93
Back out version changes to make nose tests pass
18b5da0
Remove changes to requirements
8eca097
Put nose back in requirements.txt
277c06f
Remove version bump to version.sbt
b55ee48
Remove packages related to testing
f8a8fd9
Remove old setup.py / setup.cfg
bc2cb36
New pyproject.toml and poetry.lock
728be33
Short README for Python package, poetry won't allow a ../README.md path
3cea1a8
Remove requirements files in favor of pyproject.toml
87cc975
Try to poetrize CI build
6f84a5a
pyspark min 3.4
9a8eef0
Local python README in pyproject.toml
75ecd99
Trying to remove he working folder to debug scala issue
80231d0
Set Python working directory again
2a9170b
Accidental newline
3de2263
Install Python for test...
4662717
Run tests from python/ folder
1b7b9f8
Try running tests from python/
58da493
poetry run the unit tests
9f4aa24
rjurney added 22 commits February 17, 2025 08:53
Removing SPARK_HOME
272be06
Returned SPARK_HOME settings
4587999
Minimized the PR to just these files
2422b22
Merge in rjurney/build-upgrades and in turn master
073dced
Created tutorials dependency group to minimize main bloat
0a1faba
Make motif.py execute in whole again
c0d6d7b
Minor isort format and cleanup of download.py
5bb4c26
Minor isort format and cleanup of utils.py
99e6a4d
Removed case sensitivity from the script - that was confusing people …
662e197 
…who just pasted or tried to run the code without a new SparkSession.
motif.py now matches tutorial code, runs and handles case insensitivity.
beaa35d
Regenerate poetry.lock
1bf4a9e
Setup a 'graphframes stackexchange' comand.
ef19784
Make graphframes.tutorials.motif use a checkpoint dir unique, and fro…
4400cb4 
…m SparkSession.sparkContext. Use click.echo instead of print
Use spark.sparkContext.setCheckpointDir directly instead of instantia…
d549c56 
…ting a SparkContext. print-->click.echo
Using 'from __future__ import annotations' intsead of List and Tuple
b970636
Now retry three times if we can't connect for any reason in 'graphfra…
3788941 
…mes stackexchange' command.
Merge master
e95bbbe
Merge branch 'master' of github.com:graphframes/graphframes
413a915
Merge branch 'master' of github.com:graphframes/graphframes
b6e8c27
Ignore spark-warehouse/
d4dccd3
Added urls for Pluralsight GraphFrames classes
90001ee
Trailing newline
789e318

codecov-commenter commented Jul 12, 2025 •
edited
Loading

Copy link
Copy Markdown

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.87%. Comparing base (bc487ef) to head (789e318).
Report is 44 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master     #629      +/-   ##
==========================================
- Coverage   91.43%   89.87%   -1.56%     
==========================================
  Files          18       18              
  Lines         829      909      +80     
  Branches       52      115      +63     
==========================================
+ Hits          758      817      +59     
- Misses         71       92      +21

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

SemyonSinchenko commented Jul 12, 2025

Copy link
Copy Markdown
Collaborator

@rjurney Could you please resolve conflicts?

SemyonSinchenko commented Jul 12, 2025

Copy link
Copy Markdown
Collaborator

Keep in mind please also changes to README from this PR: #626

rjurney commented Jul 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Keep in mind please also changes to README from this PR: #626

@SemyonSinchenko Okay, I merged main and resolved conflicts. These changes are small and can be made before #626.

SemyonSinchenko merged commit 191c355 into master Jul 14, 2025
10 checks passed
rjurney deleted the rjurney/readme-3 branch September 25, 2025 01:14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

SemyonSinchenko SemyonSinchenko approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL