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

Add Documentation to all analyzers #59

Open
TheShiny1 opened this issue Mar 24, 2021 · 4 comments
Open

Add Documentation to all analyzers #59

TheShiny1 opened this issue Mar 24, 2021 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation Team 4

Comments

@TheShiny1
Copy link
Collaborator

No description provided.

@enpuyou
Copy link
Contributor

enpuyou commented Mar 24, 2021

Hi @TheShiny1, can you provide a description of the issue you are raising? What kind of documentation are you considering? A documentation website like #33 suggests? Or documentation in the code?

@enpuyou enpuyou added the documentation Improvements or additions to documentation label Mar 24, 2021
@Mai1902
Copy link
Collaborator

Mai1902 commented Mar 30, 2021

Hi @enpuyou, our idea is somewhat similar to #33 since we want to add description to all analyzers and visualization. However, we want it to appear right on GatorMiner instead of developing a separated website. Do you think we should write description right on GatorMiner by adding a .txt file, scan through the file and print it under every analyzer by adding print command in visualization.py and summarizer.py or we should follow your idea of creating a website. I am so sorry that I asked it too late. But I do found the idea of writing a website is really nice.

@enpuyou
Copy link
Contributor

enpuyou commented Mar 30, 2021

@Mai1902 No worries! I think your idea is great and there is no need for creating another website. I do think you might wanna use markdown instead of txt, use st.write() instead of print(). Thanks!

@Kevin487 Kevin487 self-assigned this Mar 30, 2021
@favourojo favourojo added the Team 8 Team 8 is working on this issue label Apr 11, 2021
@Mai1902 Mai1902 added the Team 4 label Apr 20, 2021
@favourojo favourojo removed the Team 8 Team 8 is working on this issue label Apr 21, 2021
@TheShiny1
Copy link
Collaborator Author

@enpuyou I misclicked, my bad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Team 4
Projects
None yet
Development

No branches or pull requests

8 participants