Skip to content

Limit dependencies for documentation guides#242

Merged
TimothyWillard merged 1 commit intomainfrom
push-qxwlmmznyqll
Apr 27, 2026
Merged

Limit dependencies for documentation guides#242
TimothyWillard merged 1 commit intomainfrom
push-qxwlmmznyqll

Conversation

@TimothyWillard
Copy link
Copy Markdown
Collaborator

Description

  • Limited the dependencies in the environment.yaml conda environment files used in the flepimop2 documentation guides.
  • Clarified documentation in some key places.

Related issues

n/a

Checklist

  • I have read through and understand the pull request process.
  • I ran successfully ran CI local via just ci.
  • I have updated the CHANGELOG.md or noted "no major changes" in my commit if the PR is small.

@TimothyWillard TimothyWillard force-pushed the push-qxwlmmznyqll branch 2 times, most recently from 94b319a to bad4e40 Compare April 27, 2026 19:31
- Limited the dependencies in the `environment.yaml` conda environment
  files used in the `flepimop2` documentation guides.
- Clarified documentation in some key places.
Copy link
Copy Markdown
Member

@pearsonca pearsonca left a comment

Choose a reason for hiding this comment

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

Just double checking on the ipynb element - so that's exclusively in the full-feature section now? Otherwise seems fine.

Comment thread docs/index.md
flepimop2 simulate --target hires configs/config.yaml
flepimop2 process --target hires configs/config.yaml
```

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do we want chop this out entirely? Maybe hang onto Contributing

@TimothyWillard
Copy link
Copy Markdown
Collaborator Author

Just double checking on the ipynb element - so that's exclusively in the full-feature section now? Otherwise seems fine.

It was only in the full-feature section, the command to run it was never provided in docs/index.md. This PR just removes the notebook from that section and the unneeded dependency.

@TimothyWillard TimothyWillard merged commit 6f13349 into main Apr 27, 2026
4 of 6 checks passed
@TimothyWillard TimothyWillard deleted the push-qxwlmmznyqll branch April 27, 2026 20:02
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