Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switching to scikit-build-core #62

Merged
merged 9 commits into from
Jan 24, 2024
Merged

Switching to scikit-build-core #62

merged 9 commits into from
Jan 24, 2024

Conversation

ctlee
Copy link
Collaborator

@ctlee ctlee commented Jan 23, 2024

Description

scikit-build-core is a scikit-build rewrite with better support for modern python packaging. https://scikit-build-core.readthedocs.io/

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Switch python build system to scikit-build-core
  • Clean up python module install to include only pymem3dg and required libraries. (Excludes mem3dg lib and headers along with contents of dependent projects)
  • Remove outdated versioning schemes

Questions

N/A

Status

  • Ready to go

ctlee added 9 commits January 19, 2024 14:58
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
We prefer using polyscope python package for visualization

Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Switching from scikit-build to scikit-build-core workflow for better editable builds, compliance with pip install etc...

Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
Signed-off-by: Christopher T. Lee <ctlee@ucsd.edu>
@ctlee ctlee merged commit c0963e4 into development Jan 24, 2024
5 checks passed
@ctlee ctlee deleted the scikit-build-core branch January 24, 2024 22:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant