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

Release 1.0.0 #289

Open
strengejacke opened this issue Jan 7, 2025 · 13 comments
Open

Release 1.0.0 #289

strengejacke opened this issue Jan 7, 2025 · 13 comments
Labels
Help us 👀 Help is needed to implement something
Milestone

Comments

@strengejacke
Copy link
Member

strengejacke commented Jan 7, 2025

@DominiqueMakowski I think for version 1.0.0, most/all of these should be fixed, if possible.

Originally posted by @strengejacke in #287 (comment)

Tasks

Preview Give feedback
No tasks being tracked yet.
@strengejacke strengejacke added this to the 1.0.0 milestone Jan 7, 2025
@strengejacke strengejacke pinned this issue Jan 7, 2025
@DominiqueMakowski
Copy link
Member

For plotting, currently w do a lot of guesswork to pick what best goes in x / color / alpha depending on the type of the variables (factor, continuous etc). This is probably over-complex and not optimal anyways, so we could simplify the rules and streamline the inner workings. We could have the same internal for every estimate_* functions with the following aesthetics:

  • y = whatever is predicted, means, prediction, contrasts, slopes etc.
  • x = first argument of by
  • color = second argument of by
  • alpha = third argument of by
  • Everything else goes into facet_wrap

@strengejacke
Copy link
Member Author

Sounds good! x could be either first value of by, or first value of contrast or trend (if available - I think we save all the information as attributes).

@strengejacke
Copy link
Member Author

I think we should have a 0.9.0 release soon, we have added a lot of new features the last time. Which issues would you like to be fixed for 0.9.0?

@DominiqueMakowski
Copy link
Member

We can focus on bugs / broken vignettes since there's not a lot and leave plotting refactoring for 1.0 ?

@strengejacke

This comment has been minimized.

@DominiqueMakowski
Copy link
Member

true, comparison sounds good

@DominiqueMakowski
Copy link
Member

of type which is more generic

@strengejacke
Copy link
Member Author

type could be confused with the type argument in predict(). Furthermore, comparison = "(b3-b1)=(b4-b2)" indicates a comparison, but not really a type? In the particular case of contrasts, comparison seems more generic to me.

@DominiqueMakowski
Copy link
Member

agreed!

@strengejacke
Copy link
Member Author

@strengejacke strengejacke added Help us 👀 Help is needed to implement something and removed help wanted labels Jan 10, 2025
@strengejacke

This comment has been minimized.

@strengejacke
Copy link
Member Author

Test coverage increased from ~36% up to 80%

Image

@DominiqueMakowski
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help us 👀 Help is needed to implement something
Projects
None yet
Development

No branches or pull requests

2 participants