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

All features 202403 #11

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

russHyde
Copy link
Contributor

Draft branch containing all features from Jumping Rivers' development work on tfpscanner in Feb/March 2024.

This branch is just to allow Erik's team to experiment with the introduced features, while the associated code is under review. The real thing will be merged into mrc-ide/tfpscanner once the source code for individual features has been reviewed. That merge will come from dev-202403 rather than this branch.

@russHyde russHyde marked this pull request as draft March 15, 2024 14:54
@russHyde russHyde changed the title Draft all features 202403 All features 202403 Mar 15, 2024
@russHyde
Copy link
Contributor Author

Note that a {renv}-based environment is defined in here. This can be used to ensure that the output from create_browser_data() is compatible with the environment within which tfpbrowser will eventually run.

@russHyde
Copy link
Contributor Author

The CI runs work fine against Bioconductor 3.18 / R 4.3. But if an {renv} definition is present, we get failures to install system packages. Suggest that the way to ensure the figures are generated using a matching environment to the browser app is by making a figure-generating env within tfpbrowser, rather than having an in-package env here.

bump CI scripts to use R 4.3 and bioconductor 3.18
russHyde and others added 5 commits April 15, 2024 15:23
…Blue-Red diverging palette with light-grey midpoint
…han in (non-exported) save_trees()

This makes the function call for save_trees() more similar to
save_sina_plot(), where the width and height are specified as function
arguments.
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.

None yet

1 participant