Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate CytoDataFrame from coSMicQC as standalone package #4

Merged
merged 9 commits into from
Oct 30, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
49 changes: 49 additions & 0 deletions .github/ISSUE_TEMPLATE/issue.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
# GitHub Issue template for bug reports
name: Open a GitHub issue
description: >
Please use this form to send along new ideas for content or
changes that might be helpful!

body:
- type: checkboxes
attributes:
label: Is this a duplicate of an existing idea for this project?
description: >
Please make sure to search in the
[issues](https://github.com/WayScience/CytoDataFrame/issues) first
to see whether the same issue was reported already.
If you find an existing issue, please don't hesitate to comment
on it or add a reaction to existing content!
options:
- label: >
I found no existing
[issues](https://github.com/WayScience/CytoDataFrame/issues)
covering this topic.
required: true

- type: textarea
id: description
attributes:
label: What is your idea?
description: >
Please provide a specific description of what you'd like to see
including the context and what the result might look like.
placeholder: >
For example: "When x happens I see y.
The following might be a good way to address this ..."
validations:
required: true

- type: checkboxes
attributes:
label: Would you like to work on a solution for this?
description: >
This is a community-driven project and we
love new contributors (including through opening or adding to issues)!
This is an optional check to help us understand your interest to be
involved (especially if you already have a good understanding
of how to implement it).
We are happy to guide you in the contribution process and please
don't hesitate to reach out for help along the way.
options:
- label: Yes I am willing to submit a PR for this!
37 changes: 37 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
<!-- _modified from [EmbeddedArtistry](https://embeddedartistry.com/blog/2017/08/04/a-github-pull-request-template-for-your-projects/)_
_referenced with modifications from [pycytominer](https://github.com/cytomining/pycytominer/blob/master/.github/PULL_REQUEST_TEMPLATE.md)_ -->

# Description

<!--
Thank you so much for your contribution to CytoDataFrame!

Please _succinctly_ summarize your proposed change.
Namely, consider addressing the following questions:

- What motivated you to open this pull request?
- Were there any special adjustments you had to make to complete the work?
- Are there any issues which are related to this pull request (you may used a `#<digit>` to reference GitHub issues as links within this description)?

Also, if you haven't already, please use `pre-commit run --all-files` to help check your files using this project's pre-commit configuration.
Pre-commit checks will automatically run as part of opening this pull request and we seek to ensure all checks pass before merging changes.
-->

## What kind of change(s) are included?

- [ ] Documentation (changes docs or other related content)
- [ ] Bug fix (fixes an issue).
- [ ] Enhancement (adds functionality).
- [ ] Breaking change (these changes would cause existing functionality to not work as expected).

# Checklist

Please ensure that all boxes are checked before indicating that this pull request is ready for review.

- [ ] I have read and followed the [CONTRIBUTING.md](CONTRIBUTING.md) guidelines.
- [ ] I have searched for existing content to ensure this is not a duplicate.
- [ ] I have performed a self-review of these additions (including spelling, grammar, and related).
- [ ] These changes pass all pre-commit checks.
- [ ] I have added comments to my code to help provide understanding
- [ ] I have added a test which covers the code changes found within this PR
- [ ] I have deleted all non-relevant text in this pull request template.
16 changes: 16 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
# GitHub Dependabot configuration
# Note: there is no interaction between this
# configuration and dependabot security updates.
# See here for more information:
# https://docs.github.com/en/code-security/dependabot/dependabot-security-updates/about-dependabot-security-updates#about-dependabot-security-updates

version: 2
updates:
# GitHub Actions checks
# See here for more information:
# https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuring-dependabot-version-updates
- package-ecosystem: "github-actions"
directory: "/"
schedule:
# Check for updates to GitHub Actions every week
interval: "weekly"
21 changes: 21 additions & 0 deletions .github/release-drafter.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
---
# template configuration for release-drafter
# see: https://github.com/release-drafter/release-drafter
name-template: 'v$RESOLVED_VERSION'
tag-template: 'v$RESOLVED_VERSION'
version-resolver:
major:
labels:
- 'release-major'
minor:
labels:
- 'release-minor'
patch:
labels:
- 'release-patch'
default: patch
change-template: '- $TITLE (@$AUTHOR via #$NUMBER)'
template: |
## Changes

$CHANGES
23 changes: 23 additions & 0 deletions .github/workflows/draft-release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
---
# workflow for drafting releases on GitHub
# see: https://github.com/release-drafter/release-drafter
name: release drafter

on:
push:
branches:
- main

jobs:
draft_release:
permissions:
# write permission is required to create a github release
contents: write
# write permission is required for autolabeler
# otherwise, read permission is required at least
pull-requests: write
runs-on: ubuntu-latest
steps:
- uses: release-drafter/release-drafter@v6
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
50 changes: 50 additions & 0 deletions .github/workflows/publish-docs.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
---
# used for publishing documentation on push to main or published release
name: publish docs

on:
push:
branches:
- main
release:
types:
- published

jobs:
build:
# only build and deploy docs if the actor is not dependabot
if: ${{ github.actor != 'dependabot[bot]' }}
runs-on: ubuntu-22.04
steps:
- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 0
- uses: actions/setup-python@v5
with:
python-version: "3.11"
- name: Setup for poetry
run: |
python -m pip install poetry poetry-dynamic-versioning
- name: poetry deps
run: poetry install
- name: Build documentation
run: |
mkdir pages
touch pages/.nojekyll
cd docs
poetry run sphinx-build src build
# remove any doctrees dirs which aren't needed for publishing
find ./build -type d -name '.doctrees' -exec rm -rf {} +
cp -r build/* ../pages/
- name: Add index redirector to latest docs
run: |
cp docs/redirector.html pages/redirector.html
- name: Add media folder to latest docs
run: |
cp -r docs/src/media pages/media
- name: Deploy documentation
uses: JamesIves/github-pages-deploy-action@v4
with:
branch: pages
folder: pages
34 changes: 34 additions & 0 deletions .github/workflows/publish-pypi.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
---
# used for publishing packages to pypi on release
name: publish pypi release

on:
release:
types:
- published

jobs:
publish_pypi:
runs-on: ubuntu-latest
environment: release
permissions:
# IMPORTANT: this permission is mandatory for trusted publishing
id-token: write
steps:
- name: Checkout
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Python setup
uses: actions/setup-python@v5
with:
python-version: "3.11"
- name: Setup for poetry
run: |
python -m pip install poetry poetry-dynamic-versioning
- name: Install environment
run: poetry install --no-interaction --no-ansi
- name: poetry build distribution content
run: poetry build
- name: Publish package distributions to PyPI
uses: pypa/gh-action-pypi-publish@release/v1
50 changes: 50 additions & 0 deletions .github/workflows/run-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
---
# used for running tests
name: tests

on:
push:
branches: [main]
pull_request:
branches: [main]

jobs:
pre_commit_checks:
runs-on: ubuntu-22.04
steps:
# checks out the repo
- uses: actions/checkout@v4
# run pre-commit
- name: Python setup
uses: actions/setup-python@v5
with:
python-version: "3.11"
- name: Setup for poetry
run: |
python -m pip install poetry
- name: Install environment
run: poetry install --no-interaction --no-ansi
- uses: pre-commit/[email protected]
run_tests:
strategy:
matrix:
python_version: ["3.9", "3.10", "3.11", "3.12"]
os: [ubuntu-22.04, macos-13]
runs-on: ${{ matrix.os }}
env:
OS: ${{ matrix.os }}
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Python setup
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python_version }}
- name: Setup for poetry
run: |
python -m pip install poetry
- name: Install environment
run: poetry install --no-interaction --no-ansi
- name: Run pytest
# run all tests except those marked generate_report_image
run: poetry run pytest -m "not generate_report_image"
41 changes: 13 additions & 28 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -82,34 +82,13 @@ target/
profile_default/
ipython_config.py

# pyenv
# For a library or package, you might want to ignore these files since the code is
# intended to run in multiple environments; otherwise, check them in:
# .python-version

# pipenv
# According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
# However, in case of collaboration, if having platform-specific dependencies or dependencies
# having no cross-platform support, pipenv may install dependencies that don't work, or not
# install all needed dependencies.
#Pipfile.lock

# poetry
# Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
# This is especially recommended for binary packages to ensure reproducibility, and is more
# commonly ignored for libraries.
# https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
#poetry.lock

# pdm
# Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
#pdm.lock
# pdm stores project-wide configurations in .pdm.toml, but it is recommended to not include it
# in version control.
# https://pdm.fming.dev/latest/usage/project/#working-with-version-control
# https://pdm.fming.dev/#use-with-ide
.pdm.toml
.pdm-python
.pdm-build/

# PEP 582; used by e.g. github.com/David-OConnor/pyflow and github.com/pdm-project/pdm
__pypackages__/
Expand Down Expand Up @@ -154,9 +133,15 @@ dmypy.json
# Cython debug symbols
cython_debug/

# PyCharm
# JetBrains specific template is maintained in a separate JetBrains.gitignore that can
# be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
# and can be added to the global gitignore or merged into this file. For a more nuclear
# option (not recommended) you can uncomment the following to ignore the entire idea folder.
#.idea/
# test data ignores
*.tif
*.tiff
*.sqlite
*.parquet
*.zip
*.csv

.DS_Store

# jupyter notebook build files from myst-nb
docs/jupyter_execute
Loading