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

Fixes #227 ribbon transparency can be changed in all plots #228

Merged

Conversation

pchelle
Copy link
Collaborator

@pchelle pchelle commented Mar 24, 2022

While fixing this, I also fixed a typo in runThemeMaker
Besides, I found a way of centralizing and simplifying the process of defining how lines, points, ribbons are plotted

…d in all plots

While fixing this, I also fixed a typo in runThemeMaker 
Besides, I found a way of centralizing and simplifying the process of defining how lines, points, ribbons are plotted
@pchelle pchelle requested a review from IndrajeetPatil March 24, 2022 17:13
@pchelle pchelle mentioned this pull request Mar 24, 2022
pchelle added a commit to pchelle/TLF-Library that referenced this pull request Mar 24, 2022
This code was a fix committed in PR Open-Systems-Pharmacology#228 that may lead to a merge conflict
@msevestre msevestre merged commit 5379e02 into Open-Systems-Pharmacology:develop Mar 24, 2022
IndrajeetPatil pushed a commit that referenced this pull request Mar 24, 2022
* Fixes #224 plotTimeProfile and its mapping are more versatile

Now simulated and observed data can be plotted both independently or synchronized (as illustrated by vignette) with correct legend for both
I also removed lloq from ObservedDataMapping that was making things complicated

* Remove ggplot2 messages when aesthetic scales are updated

* Export function to update time profile plot legend easily

* Prevent time profile errors

This code was a fix committed in PR #228 that may lead to a merge conflict
@pchelle pchelle deleted the 227_transparency branch February 15, 2024 16:19
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