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

Added centrality procedure instructions #513

Merged
merged 8 commits into from
Mar 29, 2023
Merged

Added centrality procedure instructions #513

merged 8 commits into from
Mar 29, 2023

Conversation

zsweger
Copy link
Contributor

@zsweger zsweger commented Mar 26, 2023

Added centrality procedure instructions

@plexoos
Copy link
Member

plexoos commented Mar 27, 2023

Can we avoid adding binary files in the repository?

@zsweger
Copy link
Contributor Author

zsweger commented Mar 27, 2023

Okay we can remove those

@plexoos
Copy link
Member

plexoos commented Mar 27, 2023

Ah... we let those root files slip in #340. Since it is still the HEAD of the main branch, I think it is ok if we force remove them.
However, my comment was about the pdf file. How do you think other users can update it when needed? Can you use the .md or .rst formats instead?

@zsweger
Copy link
Contributor Author

zsweger commented Mar 28, 2023

Okay I've added a markdown version of the instructions and deleted the pdf!

nigmatkulov and others added 8 commits March 29, 2023 16:46
The commit will create a new directory that should be excluded from the
compilation process. At the first iteration it should be cross checked
(if compilation will pass the CI/CD). Also before merging Zachary Sweger
should confirm that the subdirectories contain the latest versions of
the Glauber codes.

---------

Co-authored-by: Grigory Nigmatkulov <nigmaktulov@gmail.com>
Co-authored-by: Dmitri Smirnov <dmixsmi@gmail.com>
Co-authored-by: Zachary Sweger <zsweger@berkeley.edu>
Added centrality procedure instructions
Deleting binary file for merge with main branch
Deleting other binary file
Deleted pdf version of instructions to opt for markdown version
Created markdown version of centrality instructions
missed some dollar signs
@plexoos plexoos merged commit 776be20 into main Mar 29, 2023
@plexoos plexoos deleted the zsweger-patch-2 branch March 29, 2023 20:51
@genevb genevb removed their request for review March 30, 2023 14:01
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