Skip to content

chore: move pipelines out of api package#372

Merged
ltalirz merged 4 commits into
mainfrom
chore/move-pipelines
Jun 12, 2026
Merged

chore: move pipelines out of api package#372
ltalirz merged 4 commits into
mainfrom
chore/move-pipelines

Conversation

@ltalirz

@ltalirz ltalirz commented May 29, 2026

Copy link
Copy Markdown
Contributor

pipelines used by the API package still belong in amorphouspy, since they can be used by users of the python package as well.

pipelines used by the API package still belong in `amorphouspy`, since
they can be used by users of the python package as well.
@github-actions github-actions Bot added the type: ci Changelog: CI / chore changes (Maintenance) label May 29, 2026
@codecov

codecov Bot commented May 29, 2026

Copy link
Copy Markdown

Move visualization-related functions into separate visualization/
folder.
@ltalirz

ltalirz commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

@Atilaac this would be the rough outline

At this point, the "pipelines" (except for the top-level pipeline) are still python functions themselves; this will change once I reactivate #174 and the viscosity pipeline will actually become an executorlib workflow.

Some of the current "pipelines" may also move into the code outside the pipelines folder (e.g. the code for extracting the equilibration frames

@ltalirz

ltalirz commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

now that the pipeline code moves in to the amorphouspy package, I will need to add tests for it before I can merge the PR ;-)

@ltalirz ltalirz marked this pull request as ready for review June 1, 2026 12:29
@ltalirz ltalirz marked this pull request as draft June 1, 2026 12:29
@ltalirz ltalirz added the integration Tag a PR with this to run integration tests label Jun 12, 2026
@ltalirz ltalirz marked this pull request as ready for review June 12, 2026 10:47
@ltalirz ltalirz merged commit 35946db into main Jun 12, 2026
10 of 11 checks passed
@ltalirz ltalirz deleted the chore/move-pipelines branch June 12, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration Tag a PR with this to run integration tests type: ci Changelog: CI / chore changes (Maintenance)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant