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

Update some descriptions #75

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Update some descriptions #75

merged 2 commits into from
Jun 17, 2024

Conversation

yebai
Copy link
Member

@yebai yebai commented Jun 14, 2024

Some tweaks; probably subjective. More suggestions on edits are welcome!

@yebai
Copy link
Member Author

yebai commented Jun 14, 2024

@storopoli, we should probably mention TuringGLM and your Bayes course on the landing page. Can you draft something?

@storopoli
Copy link
Member

storopoli commented Jun 14, 2024

I've added the tutorials. But I don't know where we should add TuringGLM.jl.

EDIT: I added them near the other tutorials as well.

@torfjelde
Copy link
Member

Happy with the current edits:)

Want me to approve or is this PR meant to address all the points in the list above before merging?

Copy link
Member

@storopoli storopoli left a comment

Choose a reason for hiding this comment

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

LGTM, up to you @yebai.

@yebai
Copy link
Member Author

yebai commented Jun 17, 2024

Thanks. It's better to incorporate the additional information via multiple PRs. I opened an issue to track that. #76

@yebai yebai merged commit 8bb8500 into main Jun 17, 2024
@yebai yebai deleted the yebai-patch-1 branch June 17, 2024 20:37
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.

3 participants