-
Notifications
You must be signed in to change notification settings - Fork 45
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
Overall discussion for Tip 4 #245
Comments
|
Agreed. It still feels out of place, even after it was moved (from tip 1?) and further tweaked (e.g. #234). Not sure about where it fits best but open to suggestions. |
First half of this sentence feels very hard to read. |
@pstew and @SiminaB the data simulation paragraph just jumped out at me for feeling out of place both content-wise and stylistically. I'm tempted to cut it down to a sentence or two and tuck it into an existing paragraph somewhere. The paper is already quite long and I don't think this paragraph is adding a ton of useful content. |
@Benjamin-Lee I agree. Go for it. |
@Benjamin-Lee Thanks! Approved! |
This is to discuss outstanding issues for Tip 4: Know your data and your question.
https://github.com/Benjamin-Lee/deep-rules/blob/master/content/06.know-your-problem.md
The text was updated successfully, but these errors were encountered: