Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Try usingsysconfig to configure the SS JIT#515

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
eriknw wants to merge8 commits intopython-graphblas:main
base:main
Choose a base branch
Loading
fromeriknw:sysconfig_for_jit

Conversation

eriknw
Copy link
Member

No description provided.

@codecov
Copy link

codecovbot commentedOct 19, 2023
edited
Loading

Codecov Report

Merging#515 (72d6154) intomain (228faa1) willdecrease coverage by61.69%.
The diff coverage is83.33%.

@@             Coverage Diff             @@##             main     #515       +/-   ##===========================================- Coverage   91.65%   29.96%   -61.69%===========================================  Files          95       95                 Lines       21254    21265       +11       Branches     3987     3990        +3     ===========================================- Hits        19480     6373    -13107- Misses       1584    14499    +12915- Partials      190      393      +203

@eriknweriknw marked this pull request as draftOctober 19, 2023 02:16
@eriknw
Copy link
MemberAuthor

Lessons learned:

  • Usingsysconfig to provide defaults appears to "just work" for Ubuntu and Mac
    • It also appears to mostly work on Windows except when usingpython-suitesparse-graphblas wheels (which may not play nicely with conda-forge)
  • pandas 2.1 and numpy 1.21 can't be installed together (note to myself)

So, usingsysconfig seem like sane defaults to me (much more sane than using build-time info). How should we set this for Python users? Should we set this by default inpython-suitesparse-graphblas, and, if so, should we save the original config and make it easy to restore?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

1 participant
@eriknw

[8]ページ先頭

©2009-2025 Movatter.jp