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

fix README #54

Merged
merged 4 commits into from
Jul 21, 2023
Merged

fix README #54

merged 4 commits into from
Jul 21, 2023

Conversation

rnmitchell
Copy link
Contributor

Minor typos in README

@rnmitchell
Copy link
Contributor Author

Not sure how I did it, but it seems to be fixed now even though it's not showing up as a difference? (I think likely because I made the change on the master branch before creating a new branch).

There was another sentence I removed as well. But I think this is ready for review and merge @standage

@rnmitchell
Copy link
Contributor Author

wait hold on a sec. Not ready yet!

@@ -61,7 +61,6 @@ def create_sample_df(indiv_df, output_type, all_col):
.unstack(0)
.reset_index()
)
compiled_table.to_csv("test.csv", index=False)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoopsies, left in some debugging code 😄

@rnmitchell
Copy link
Contributor Author

Since I somehow completely skipped any information about the SNP workflow... I rewrote a lot of the README to be more informative about the SNP workflow.

This is now ready @standage (I hope!).

@standage standage merged commit cfb2710 into master Jul 21, 2023
@standage standage deleted the readme branch July 21, 2023 18:31
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