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

Hex logo #791

Open
sbfnk opened this issue Sep 24, 2024 · 13 comments
Open

Hex logo #791

sbfnk opened this issue Sep 24, 2024 · 13 comments

Comments

@sbfnk
Copy link
Contributor

sbfnk commented Sep 24, 2024

Do we want one? Here's an initial attempt.

image
@seabbs
Copy link
Contributor

seabbs commented Sep 24, 2024

Yes I think we do. Could maybe make the same sort of thing but horizontal?

@sbfnk
Copy link
Contributor Author

sbfnk commented Sep 25, 2024

e.g.?

image

@athowes
Copy link

athowes commented Sep 25, 2024

e.g.?

I think this one is good. If possible it could be nice to have some elements of stylistic similiarity between all Epiforecasts / epinowcast packages.

@jamesmbaazam
Copy link
Contributor

Is this an svg? I could play around with some options.

@jamesmbaazam
Copy link
Contributor

I feel like the white space below the plot is a bit awkward so the plot should probably sit on the bottom line.

@sbfnk
Copy link
Contributor Author

sbfnk commented Sep 25, 2024

SVG versions at:

https://u.pcloud.link/publink/show?code=XZpcPI0Z49hkS15tubLOWqpHYvjHUh1nEu07
https://u.pcloud.link/publink/show?code=XZ8cPI0ZCsnuPnSgDW8IjxTHkzr0qmEiHbt7

Alternative would be to create it from a script as e.g. in https://github.com/epinowcast/epidist/blob/main/inst/make_hexsticker.R which would probably help with the stylistic continuity that @athowes has suggested.

@seabbs
Copy link
Contributor

seabbs commented Oct 2, 2024

I think I prefer the results we get from hexsticker but I don't feel strongly at all

@seabbs
Copy link
Contributor

seabbs commented Oct 2, 2024

See epinowcast/epidist#363 for a very critical whitespace issue this might run into and a solution from @athowes.

@athowes
Copy link

athowes commented Oct 2, 2024

See epinowcast/epidist#363 for a very critical whitespace issue this might run into and a solution from @athowes.

Btw this works best if you increase the dpi up to 1200 then use the solution (if lower it trims it in an ugly way).

@jamesmbaazam
Copy link
Contributor

What's the progress on this?

@athowes
Copy link

athowes commented Oct 7, 2024

I am in favour of the "make a script similar to https://github.com/epinowcast/epidist/blob/main/inst/make_hexsticker.R" solution.

@sbfnk
Copy link
Contributor Author

sbfnk commented Oct 7, 2024

I agree

@seabbs
Copy link
Contributor

seabbs commented Oct 8, 2024

To return to content I like the idea of showing the reconstructed cases by infection date as that is the USP of the package vs alternatives and something that is important to communicate (even though say Rt might be tempting)

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

No branches or pull requests

4 participants