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

Removed an extra link to the installation guide in docs #362

Merged
merged 1 commit into from
Nov 12, 2020

Conversation

levinwil
Copy link
Collaborator

Reference issue

Type of change

What does this implement/fix?

Additional information

Copy link
Member

@PSSF23 PSSF23 left a comment

Choose a reason for hiding this comment

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

@levinwil Sorry for the little-late notice, but sample_size_experiment should be included as well?

@levinwil
Copy link
Collaborator Author

levinwil commented Nov 11, 2020

@PSSF23 I don't understand what you are asking. Are you asking if it should be included?

@PSSF23
Copy link
Member

PSSF23 commented Nov 11, 2020

@levinwil Yes, I mean this new tutorial: https://github.com/neurodata/ProgLearn/blob/staging/docs/tutorials/sample_size_experiment.ipynb

Should it be included in the rst list as well?

@levinwil
Copy link
Collaborator Author

@PSSF23 No I don't think so. It's not a tutorial for ProgLearn, so I don't think we should display it in the docs. But it still needs to live here, because jovo wants it to.

@PSSF23 PSSF23 merged commit 3efec08 into staging Nov 12, 2020
@PSSF23 PSSF23 deleted the docs_repetition_minimization branch November 12, 2020 16:39
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