Skip to content

FEAT: cooccurence network integration, github action docker image build and deployment - #1

Merged
laschuet merged 5 commits into
mainfrom
feature-cooccurence
Apr 1, 2026
Merged

FEAT: cooccurence network integration, github action docker image build and deployment#1
laschuet merged 5 commits into
mainfrom
feature-cooccurence

Conversation

@Mariuuus

@Mariuuus Mariuuus commented Mar 30, 2026

Copy link
Copy Markdown
Collaborator

FEAT: integrating co-occurrence networks

Summary

Adjusts dependencies and imports to allow for an integration of the (previously) separated co-occurrence network analysis.

Changes

  • Added .gitignore
  • Downgraded numpy and pandas due to the compatibility with grakel
  • Added missing dependency for openpyxl
  • Added Dockerfile (uv-based) and adjust workflow (build and deploy image)

How to Test

  1. Check out this branch
  2. Run uv run westerfeld/cooccurence.py (make sure you provided the input files)

Results are located at out

Docker

  1. Check out this branch
  2. (Manual build) docker build -t westerfeld-analyses -f docker/Dockerfile .
  3. Run docker container (mounting data & output folder):
    sudo docker run \
          -v ../data:/app/input \
          -v .out:/app/out \
          westerfeld-analyses  westerfeld/cooccurence.py
    

Notes

The downgrade of numpy and pandas due to the compatibility with grakel may come with some downsides.
Currently there is no (official) newer grakel version that supports modern numpy and the PR targeting this issue is on open for 8 months.

@Mariuuus
Mariuuus marked this pull request as draft March 30, 2026 15:03
@Mariuuus Mariuuus self-assigned this Mar 30, 2026
@Mariuuus
Mariuuus force-pushed the feature-cooccurence branch 2 times, most recently from 24e9802 to 99ec2c0 Compare March 30, 2026 15:43
@Mariuuus Mariuuus changed the title Feature cooccurence FEAT: cooccurence network integration, github action docker image build and deployment Mar 30, 2026
@laschuet
laschuet marked this pull request as ready for review March 30, 2026 20:04
@laschuet
laschuet marked this pull request as draft March 30, 2026 20:05
@Mariuuus
Mariuuus force-pushed the feature-cooccurence branch from a44b54e to e53221f Compare March 31, 2026 12:26
@Mariuuus
Mariuuus marked this pull request as ready for review March 31, 2026 12:32
@Mariuuus
Mariuuus requested a review from laschuet March 31, 2026 12:33

@laschuet laschuet left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I'll add a few changes myself later on. Thank you very much for this PR!

@laschuet
laschuet merged commit 2aacf80 into main Apr 1, 2026
1 check passed
@laschuet
laschuet deleted the feature-cooccurence branch April 1, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants