-
Notifications
You must be signed in to change notification settings - Fork 61
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
Repo readme #682
Repo readme #682
Conversation
I created this as a draft because the way I implemented the auto-README -> docs page copy won't work with how our travis github pages integration is implemented. However, the actual content is ready for review. |
README.md
Outdated
|
||
`Experiment` (create features and models) -> `Audition` (pick the best models) -> `Postmodeling` (dive into best models) | ||
|
||
## Documentation Quick Links |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe just a quick sentence about what each of these is (e.g., "Start here if you want...")?
installed directly using `easy_install` or `pip` (`pip install triage`), or named as a | ||
dependency of another package as `triage`. | ||
|
||
### Design an Experiment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe briefly describe what an experiment is in terms of triage (and maybe why we call it that) since it's not necessarily familiar/standard terminology.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we kind of do this later, perhaps it makes sense to move 'Triage Phase Details' up before this part? Or even just stick this section into the 'Experiment' part of the aforementioned 'Triage Phase Details'?
README.md
Outdated
To use Triage, you first need: | ||
|
||
- Python 3.6 | ||
- A PostgreSQL database with your source data (events, geographical data, etc) loaded. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a sense of the minimum version requirement for postgres?
README.md
Outdated
tasks, such as feature generation, classifier training, evaluation, and | ||
list generation. These tasks are complicated in their own right, but in | ||
addition have to be combined in different ways throughout the course of | ||
the project. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like somewhere here we should mention the focus on problems with a time component. I don't think Triage would work without any sort of temporal config, at least as it's currently built, would it?
README.md
Outdated
experiment = SingleThreadedExperiment( | ||
config=experiment_config, # a dictionary | ||
db_engine=create_engine(...), # http://docs.sqlalchemy.org/en/latest/core/engines.html | ||
project_path='/path/to/directory/to/save/data' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(maybe add a comment that this can be in S3?)
README.md
Outdated
|
||
If you're new to Triage Experiments, check out the [Dirty Duck tutorial](https://dssg.github.io/dirtyduck). It's a guided tour through Triage functionality using a real-world problem. | ||
|
||
If you're familiar with creating an Experiment but want to see more reference documentation and some deep dives, check out the links on the side. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"links on the side" in dirty duck?
README.md
Outdated
dependency of another package as `triage`. | ||
|
||
|
||
## Experiment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe make this header: Experiment: Create features and Models
README.md
Outdated
|
||
If you're familiar with creating an Experiment but want to see more reference documentation and some deep dives, the [Triage Documentation Site](https://dssg.github.io/triage) has more content. | ||
|
||
## Audition |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe: Audition: Pick the Best Models
README.md
Outdated
|
||
To get started with Audition, check out its [README](https://github.com/dssg/triage/tree/master/src/triage/component/audition) | ||
|
||
## Postmodeling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe: Postmodeling: Dive Deeper into Selected Models
made a couple suggestions on the headers and might even consider numbering them ("0. Prerequisites", "1. Install", etc) unless that seems like too much? Just trying to make the flow of the pipeline super-clear at a glance. |
Codecov Report
@@ Coverage Diff @@
## master #682 +/- ##
==========================================
- Coverage 82.3% 82.26% -0.05%
==========================================
Files 95 95
Lines 6625 6625
==========================================
- Hits 5453 5450 -3
- Misses 1172 1175 +3
Continue to review full report at Codecov.
|
@shaycrk what do you think about merging this? I'm inclined to leave merge decisions up to you and @rayidghani but this seems like it's ready to go now and would add value. |
No description provided.