Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

Borda
Copy link
Contributor

@Borda Borda commented Aug 4, 2025

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

addressing long installation time...

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@Borda Borda requested a review from ethanwharris as a code owner August 4, 2025 17:42
@Borda Borda requested a review from Copilot August 4, 2025 17:43
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69%. Comparing base (1581765) to head (58f2c15).

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #393   +/-   ##
===================================
- Coverage    72%    69%   -2%     
===================================
  Files         2      2           
  Lines       441    441           
===================================
- Hits        316    306   -10     
- Misses      125    135   +10     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses long installation times for PyTorch Geometric dependencies by adding the --prefer-binary pip option to the requirements file.

  • Adds --prefer-binary flag to avoid compiling packages from source
  • Includes explanatory comments about the purpose of the change

@Borda
Copy link
Contributor Author

Borda commented Aug 4, 2025

seems it does not have desired effect :(

@Borda Borda closed this Aug 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant