Skip to content

Commit 0f520bc

Browse files
authored
Merge branch 'master' into resurrect_systematics
2 parents 9955f3f + 4b9147f commit 0f520bc

File tree

108 files changed

+5680
-8333
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

108 files changed

+5680
-8333
lines changed

Diff for: .github/dependabot.yml

+3
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,6 @@ updates:
55
directory: "/"
66
schedule:
77
interval: "daily"
8+
commit-message:
9+
# Prefix all commit messages with "npm: "
10+
prefix: "ci: "

Diff for: .github/workflows/ci.yml

+30-30
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ jobs:
2626
name: pre-commit
2727
runs-on: ubuntu-latest
2828
steps:
29-
- uses: actions/checkout@v3
30-
- uses: actions/setup-python@v4
29+
- uses: actions/checkout@v4
30+
- uses: actions/setup-python@v5
3131
- uses: pre-commit/[email protected]
3232
with:
3333
extra_args: --hook-stage manual --all-files
@@ -45,13 +45,13 @@ jobs:
4545
name: test coffea (${{ matrix.os }}) - python ${{ matrix.python-version }}, JDK${{ matrix.java-version }}
4646

4747
steps:
48-
- uses: actions/checkout@v3
48+
- uses: actions/checkout@v4
4949
- name: Set up Python ${{ matrix.python-version }}
50-
uses: actions/setup-python@v4
50+
uses: actions/setup-python@v5
5151
with:
5252
python-version: ${{ matrix.python-version }}
5353
- name: Set up JDK ${{ matrix.java-distribution }}/${{ matrix.java-version }}
54-
uses: actions/setup-java@v3
54+
uses: actions/setup-java@v4
5555
with:
5656
java-version: ${{ matrix.java-version }}
5757
distribution: ${{ matrix.java-distribution }}
@@ -67,9 +67,9 @@ jobs:
6767
# mltool installs
6868
python -m pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu
6969
python -m pip install xgboost
70-
python -m pip install tritonclient[grpc,http]
70+
python -m pip install 'tritonclient[grpc,http]!=2.41.0'
7171
# install checked out coffea
72-
python -m pip install -q -e '.[dev,parsl,dask,spark]'
72+
python -m pip install -q -e '.[dev,parsl,dask,spark]' --upgrade --upgrade-strategy eager
7373
python -m pip list
7474
java -version
7575
- name: Install dependencies (MacOS)
@@ -80,7 +80,7 @@ jobs:
8080
python -m pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu
8181
python -m pip install xgboost
8282
# install checked out coffea
83-
python -m pip install -q -e '.[dev,dask,spark]'
83+
python -m pip install -q -e '.[dev,dask,spark]' --upgrade --upgrade-strategy eager
8484
python -m pip list
8585
java -version
8686
- name: Install dependencies (Windows)
@@ -91,18 +91,18 @@ jobs:
9191
python -m pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cpu
9292
python -m pip install xgboost
9393
# install checked out coffea
94-
python -m pip install -q -e '.[dev,dask]'
94+
python -m pip install -q -e '.[dev,dask]' --upgrade --upgrade-strategy eager
9595
python -m pip list
9696
java -version
9797
9898
- name: Start triton server with example model
9999
if: matrix.os == 'ubuntu-latest'
100100
run: |
101-
docker run -d --rm -p 8000:8000 -p 8001:8001 -p 8002:8002 -v ${{ github.workspace }}/tests/samples/triton_models_test:/models nvcr.io/nvidia/tritonserver:23.04-py3 tritonserver --model-repository=/models
101+
docker run -d --rm -p 8000:8000 -p 8001:8001 -p 8002:8002 -v ${{ github.workspace }}/tests/samples/triton_models_test:/models nvcr.io/nvidia/tritonserver:23.04-pyt-python-py3 tritonserver --model-repository=/models
102102
103103
- name: Test with pytest
104104
run: |
105-
pytest --cov-report=xml --cov=coffea tests
105+
pytest --cov-report=xml --cov=coffea --deselect=test_taskvine
106106
- name: Upload codecov
107107
if: matrix.os == 'ubuntu-latest' && matrix.python-version == 3.11
108108
uses: codecov/codecov-action@v3
@@ -119,48 +119,48 @@ jobs:
119119
touch build/html/.nojekyll
120120
- name: Deploy documentation
121121
if: github.event_name == 'push' && matrix.os == 'ubuntu-latest' && matrix.python-version == 3.11
122-
uses: crazy-max/ghaction-github-pages@v3
122+
uses: crazy-max/ghaction-github-pages@v4
123123
with:
124124
target_branch: gh-pages
125125
build_dir: docs/build/html
126126
env:
127127
GH_PAT: ${{ secrets.GITHUB_OAUTH }}
128128

129-
testwq:
129+
test-vine:
130130
runs-on: ubuntu-latest
131131
needs: pre-commit
132132
strategy:
133133
matrix:
134134
python-version: ["3.11"]
135-
name: test coffea-workqueue
136-
135+
name: test coffea-taskvine
137136
steps:
138-
- uses: actions/checkout@v3
137+
- uses: actions/checkout@v4
139138
- name: Set up Conda
140-
uses: conda-incubator/setup-miniconda@v2
139+
uses: conda-incubator/setup-miniconda@v3
141140
env:
142141
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
143142
with:
144143
auto-update-conda: true
145144
python-version: ${{ matrix.python-version }}
146145
channels: conda-forge
147-
- name: Test work_queue
146+
- name: Construct conda environment
148147
shell: bash -l {0}
149148
run: |
150-
conda create --yes --name coffea-env -c conda-forge python=${{ matrix.python-version }} ndcctools dill conda-pack conda
151-
conda activate coffea-env
152-
python -m pip install --ignore-installed .
153-
cd tests
154-
conda-pack --output coffea-env.tar.gz
155-
python wq.py coffea-env.tar.gz
149+
conda create --yes --prefix ./coffea-conda-test-env -c conda-forge python=${{ matrix.python-version }} ndcctools
150+
conda activate ./coffea-conda-test-env
151+
python -m pip install .
152+
python -m pip install pytest
153+
- name: Test with pytest
154+
run: |
155+
conda run --prefix ./coffea-conda-test-env pytest tests/test_taskvine.py
156156
157157
# testskyhookjob:
158158
# runs-on: ubuntu-latest
159159
# needs: pre-commit
160160
# name: test coffea-skyhook-job
161161
#
162162
# steps:
163-
# - uses: actions/checkout@3.2.0
163+
# - uses: actions/checkout@3
164164
# - name: Test Coffea Skyhook Bindings
165165
# shell: bash -l {0}
166166
# run: |
@@ -178,30 +178,30 @@ jobs:
178178
release:
179179
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')
180180
runs-on: ubuntu-latest
181-
needs: [test, testwq]
181+
needs: [test, test-vine]
182182
strategy:
183183
matrix:
184184
python-version: ["3.11"]
185185
name: deploy release
186186

187187
steps:
188-
- uses: actions/checkout@v3
188+
- uses: actions/checkout@v4
189189
- name: Set up Python ${{ matrix.python-version }}
190-
uses: actions/setup-python@v4
190+
uses: actions/setup-python@v5
191191
with:
192192
python-version: ${{ matrix.python-version }}
193193
- name: Build package for PyPI
194194
run: |
195195
python -m pip install pip hatch --upgrade
196196
python -m hatch build -t sdist -t wheel
197197
- name: Publish package to PyPI
198-
uses: pypa/[email protected].8
198+
uses: pypa/[email protected].11
199199
with:
200200
user: __token__
201201
password: ${{ secrets.PYPI_TOKEN }}
202202

203203
pass:
204-
needs: [test, testwq]
204+
needs: [test, test-vine]
205205
runs-on: ubuntu-latest
206206
steps:
207207
- run: echo "All jobs passed"

Diff for: .github/workflows/pr.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@ jobs:
1717
name: Validate PR title
1818
runs-on: ubuntu-latest
1919
steps:
20-
- uses: amannn/action-semantic-pull-request@v5.2.0
20+
- uses: amannn/action-semantic-pull-request@v5
2121
env:
2222
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Diff for: .pre-commit-config.yaml

+9-9
Original file line numberDiff line numberDiff line change
@@ -5,26 +5,26 @@ ci:
55
for more information, see https://pre-commit.ci
66
autofix_prs: true
77
autoupdate_branch: ''
8-
autoupdate_commit_msg: '[pre-commit.ci] pre-commit autoupdate'
8+
autoupdate_commit_msg: 'ci(pre-commit): pre-commit autoupdate'
99
autoupdate_schedule: weekly
1010
skip: []
1111
submodules: false
1212

1313
repos:
1414
- repo: https://github.com/psf/black
15-
rev: 23.7.0
15+
rev: 23.12.1
1616
hooks:
1717
- id: black
1818

1919
- repo: https://github.com/PyCQA/isort
20-
rev: 5.12.0
20+
rev: 5.13.2
2121
hooks:
2222
- id: isort
2323
language_version: python3
2424
args: ["--profile", "black", "--filter-files"]
2525

2626
- repo: https://github.com/pre-commit/pre-commit-hooks
27-
rev: v4.4.0
27+
rev: v4.5.0
2828
hooks:
2929
- id: check-case-conflict
3030
- id: check-merge-conflict
@@ -37,24 +37,24 @@ repos:
3737
- id: trailing-whitespace
3838

3939
- repo: https://github.com/asottile/pyupgrade
40-
rev: v3.9.0
40+
rev: v3.15.0
4141
hooks:
4242
- id: pyupgrade
4343
args: ["--py38-plus"]
4444

4545
- repo: https://github.com/asottile/setup-cfg-fmt
46-
rev: v2.4.0
46+
rev: v2.5.0
4747
hooks:
4848
- id: setup-cfg-fmt
4949

5050
- repo: https://github.com/pycqa/flake8
51-
rev: 6.0.0
51+
rev: 7.0.0
5252
hooks:
5353
- id: flake8
5454
exclude: coffea/processor/templates
5555

5656
- repo: https://github.com/codespell-project/codespell
57-
rev: v2.2.5
57+
rev: v2.2.6
5858
hooks:
5959
- id: codespell
60-
args: ["--skip=*.ipynb","-L hist,Hist,nd,SubJet,subjet,Subjet,PTD,ptd,fPt,fpt"]
60+
args: ["--skip=*.ipynb","-L hist,Hist,nd,SubJet,subjet,Subjet,PTD,ptd,fPt,fpt,Ser,ser,hda"]

Diff for: CITATION.cff

+47
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
cff-version: 1.2.0
2+
message: "If you use this software, please cite it as below."
3+
authors:
4+
- family-names: "Gray"
5+
given-names: "Lindsey"
6+
affiliation: "Fermilab"
7+
orcid: "https://orcid.org/0000-0002-6408-4288"
8+
9+
- family-names: "Smith"
10+
given-names: "Nicholas"
11+
affiliation: "Fermilab"
12+
orcid: "https://orcid.org/0000-0002-0324-3054"
13+
14+
- family-names: "Novak"
15+
given-names: "Andrzej"
16+
affiliation: "RWTH Aachen"
17+
orcid: "https://orcid.org/0000-0002-0389-5896"
18+
- family-names: "Fackeldey"
19+
given-names: "Peter"
20+
affiliation: "RWTH Aachen University"
21+
orcid: "https://orcid.org/0000-0003-4932-7162"
22+
23+
- family-names: "Tovar"
24+
given-names: "Benjamin"
25+
affiliation: "University of Notre Dame"
26+
orcid: "https://orcid.org/0000-0002-5294-2281"
27+
28+
- family-names: "Chen"
29+
given-names: "Yi-Mu"
30+
affiliation: "University of Maryland, College Park"
31+
orcid: "https://orcid.org/0000-0002-5795-4783"
32+
33+
- family-names: "Watts"
34+
given-names: "Gordon"
35+
affiliation: "University of Washington"
36+
orcid: "https://orcid.org/0000-0002-0753-7308"
37+
38+
- family-names: "Krommydas"
39+
given-names: "Iason"
40+
affiliation: "Rice University"
41+
orcid: "https://orcid.org/0000-0001-7849-8863"
42+
43+
title: "coffea"
44+
version: 0.7.21
45+
doi: 10.5281/zenodo.7733568
46+
date-released: 2023-03-14
47+
url: "https://github.com/CoffeaTeam/coffea"

Diff for: README.rst

+9-2
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ coffea - Columnar Object Framework For Effective Analysis
2020
:target: https://anaconda.org/conda-forge/coffea
2121

2222
.. image:: https://badges.gitter.im/CoffeaTeam/coffea.svg
23-
:target: https://gitter.im/coffea-hep
23+
:target: https://matrix.to/#/#coffea-hep_community:gitter.im
2424

2525
.. image:: https://mybinder.org/badge_logo.svg
2626
:target: https://mybinder.org/v2/gh/CoffeaTeam/coffea/master?filepath=binder/
@@ -81,10 +81,17 @@ The following are installed automatically when you install coffea with pip:
8181
- `numba <https://numba.pydata.org/>`__ just-in-time compilation of python functions;
8282
- `scipy <https://scipy.org/scipylib/index.html>`__ for many statistical functions;
8383
- `matplotlib <https://matplotlib.org/>`__ as a plotting backend;
84-
- and other utility packages, as enumerated in ``setup.py``.
84+
- and other utility packages, as enumerated in ``pyproject.toml``.
8585

8686
.. inclusion-marker-3-do-not-remove
8787
8888
Documentation
8989
=============
9090
All documentation is hosted at https://coffeateam.github.io/coffea/
91+
92+
Citation
93+
========
94+
If you would like to cite this code in your work, you can use the zenodo DOI indicated in ``CITATION.cff``, or the `latest DOI <https://zenodo.org/badge/latestdoi/159673139>`__. You may also cite the proceedings:
95+
96+
- "N. Smith et al 2020 EPJ Web Conf. 245 06012"
97+
- "L. Gray et al 2023 J. Phys.: Conf. Ser. 2438 012033"

0 commit comments

Comments
 (0)