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

Fix/documentation workflow #314

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix/documentation workflow #314

wants to merge 3 commits into from

Conversation

rugeli
Copy link
Collaborator

@rugeli rugeli commented Feb 4, 2025

Problem

What is the problem this work solves, including
closes #310

Solution

What I/we did to solve this problem

  • updated documentation workflow to use deploy-pages

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Steps to Verify:

  1. Merge to main
  2. the new github action should run
  3. (We can do tests/fixes if any errors up)

Copy link
Contributor

github-actions bot commented Feb 4, 2025

Packing analysis report

Analysis for packing results located at cellpack/tests/outputs/test_spheres/spheresSST

Ingredient name Encapsulating radius Average number packed
ext_A 25 236.0

Packing image

Packing image

Distance analysis

Expected minimum distance: 50.00
Actual minimum distance: 50.01

Ingredient key Pairwise distance distribution
ext_A Distance distribution ext_A

@mogres
Copy link
Collaborator

mogres commented Feb 5, 2025

I test ran the github action by temporarily adding fix/documentation-workflow to the protection rules. The deployment ran successfully so we should be ok to merge.
image

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.

Update github action to deploy documentations
2 participants