Skip to content

Conversation

@sayalaruano
Copy link
Collaborator

@sayalaruano sayalaruano commented Jan 29, 2025

I removed the MicW2Graph example data and replaced it with a case study from the Earth Microbiome project (EMP). I changed the documentation of how to run vuegen with this update on the REDAME, and also updated the CI script. Please, take a look at this PR to check if everything is ok, or if I missed something.

@sayalaruano sayalaruano requested a review from enryH January 29, 2025 13:26
@sayalaruano sayalaruano changed the title EMP case study EMP case study as example data Jan 29, 2025
@enryH
Copy link
Collaborator

enryH commented Jan 29, 2025

I removed the old demo. I see that you subsample data. Do you maybe want to set a seed, otherwise local execution always leads to a diff. On the other hand by that you always see changing docs. Maybe that is good, but probably not:)

@sayalaruano
Copy link
Collaborator Author

Yes, it's a good idea to add a seed for the subsample, I will add it in the next commit.

…with link to vuegen and mona on the html, revealjs, pdf, and streamlit reports
…d to subsample code to create the same dataframe examples at every run
@sayalaruano sayalaruano merged commit a787ced into main Jan 30, 2025
11 checks passed
@sayalaruano sayalaruano deleted the emp-casestudy branch January 30, 2025 12:09
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