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

Repo readme #682

Merged
merged 11 commits into from
Dec 11, 2019
Merged

Repo readme #682

merged 11 commits into from
Dec 11, 2019

Conversation

thcrock
Copy link
Contributor

@thcrock thcrock commented May 3, 2019

No description provided.

@thcrock thcrock requested a review from rayidghani May 3, 2019 22:02
@thcrock
Copy link
Contributor Author

thcrock commented May 3, 2019

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.

@rayidghani rayidghani marked this pull request as ready for review June 6, 2019 18:26
README.md Outdated

`Experiment` (create features and models) -> `Audition` (pick the best models) -> `Postmodeling` (dive into best models)

## Documentation Quick Links
Copy link
Contributor

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
Copy link
Contributor

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.

Copy link
Contributor Author

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.
Copy link
Contributor

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.
Copy link
Contributor

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'
Copy link
Contributor

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.
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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

@shaycrk
Copy link
Contributor

shaycrk commented Jun 13, 2019

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-io
Copy link

codecov-io commented Jul 12, 2019

Codecov Report

Merging #682 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            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
Impacted Files Coverage Δ
src/triage/tracking.py 83.83% <0%> (-3.04%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18368c1...60207bb. Read the comment docs.

@thcrock
Copy link
Contributor Author

thcrock commented Aug 20, 2019

@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.

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