Skip to content

Commit

Permalink
Merge pull request #216 from atlanticwave-sdx/use-flit
Browse files Browse the repository at this point in the history
Use flit
  • Loading branch information
sajith authored Aug 20, 2024
2 parents dfd4205 + 586e32e commit ff08f0e
Show file tree
Hide file tree
Showing 5 changed files with 19 additions and 17 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,10 @@ jobs:
fail-fast: false
matrix:
python-version:
- "3.8"
- "3.9"
- "3.10"
- "3.11"
- "3.12"

steps:
- name: Check out code
Expand All @@ -51,7 +51,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install .[test,pygraphviz,lint]
pip install .[test,pygraphviz,lint] flit
- name: Lint with ruff
run: ruff check $(git ls-files "*.py")
Expand All @@ -63,6 +63,10 @@ jobs:
# Generate LCOV format coverage data for coveralls.
python -m coverage lcov -o coverage.lcov
- name: Build packages
run: |
python -m flit build
- name: Send coverage data to coveralls.io
uses: coverallsapp/github-action@v2
with:
Expand All @@ -73,7 +77,7 @@ jobs:
finalize:
name: finalize
needs: test
runs-on: ubuntu-latest
runs-on: ubuntu-latest
if: ${{ always() }}
steps:
- name: Indicate completion to coveralls.io
Expand Down
2 changes: 2 additions & 0 deletions .mailmap
Original file line number Diff line number Diff line change
@@ -1,2 +1,4 @@
Yifei Wang <yifei666@umd.edu> <43050198+yifei666@users.noreply.github.com>
Sajith Sasidharan <sajith@hcoop.net> <sajith@renci.org>
Italo Valcy <italo@ampath.net> <italovalcy@gmail.com>
Muhammad Usman Aziz <muhaziz@fiu.edu> <maziz@Muhammads-MacBook-Pro.local>
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,6 @@ $ source venv/bin/activate
$ pip install .[test]
```

Please note that editable installs do not work currently, due to the
shared top-level `sdx` module in datamodel.

PCE can read topology data from Graphviz dot files, if the optional
pygraphviz dependency is installed with:

Expand Down
19 changes: 9 additions & 10 deletions pyproject.toml
Original file line number Diff line number Diff line change
@@ -1,21 +1,20 @@
[build-system]
requires = [
"setuptools >= 61.0",
"setuptools-scm >= 6.2",
"wheel",
]
build-backend = "setuptools.build_meta"
requires = ["flit_core >=3.2,<4"]
build-backend = "flit_core.buildapi"

[project]
name = "sdx-pce"
version = "3.0.0.dev0"
description = "Heuristic and Optimal Algorithms for CSP and TE Computation"
dynamic = [ "version" ]
authors = [
{ name = "Yufeng Xin", email = "yxin@renci.org" },
{ name = "Yifei Wang", email = "ywang13@renci.org" }
{ name = "Yifei Wang", email = "ywang13@renci.org" },
{ name = "Cong Wang", email = "cwang@renci.org" },
{ name = "Italo Valcy", email = "italo@ampath.net" },
{ name = "Sajith Sasidharan", email = "sajith@hcoop.net" },
]
readme = "README.md"
requires-python = ">=3.8"
requires-python = ">=3.9"
license = {file = "LICENSE"}
classifiers = [
"Programming Language :: Python :: 3",
Expand All @@ -30,7 +29,7 @@ dependencies = [
"prtpy",
"pydot",
"dataclasses-json",
"sdx-datamodel @ git+https://github.com/atlanticwave-sdx/datamodel@main",
"sdx-datamodel @ git+https://github.com/atlanticwave-sdx/datamodel@3.0.0.dev0",
]

[project.urls]
Expand Down
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
[tox]
env_list =
py38
py39
py310
py311
py312

skip_missing_interpreters = true

Expand Down

0 comments on commit ff08f0e

Please sign in to comment.