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
Make travis only run on changed dirs #85
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
henryre
approved these changes
Aug 13, 2019
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.
Awesome!!
ajratner
pushed a commit
that referenced
this pull request
Aug 13, 2019
* Make travis only run on changed dirs * Small fix * Add space
ajratner
added a commit
that referenced
this pull request
Aug 14, 2019
* First rev on lighter-weight intro tutorial * Fixing @brahmaneya edits in PR * Editing down code to minimal form as suggested by HE * Simplifying code; adding TF example; editing * Editing pass over text * Transfer tags w jupytext build, minor edits * Filtering abstain values * Added a stub for SFs * Trying to fix style check * Skipping env_* files in flake * Style fixes * PR changes requested by @henryre * Quieting nltk output * Moved to getting_started * Silencing LogReg warning * Forgot to sync style fix... * Spelling fix * Addressing comments * First rev on lighter-weight intro tutorial * Fixing @brahmaneya edits in PR * Hot fix LF names (#63) * Mtl updates (#41) * [EASY] Update Scorer import paths (#58) * Save MTL updates in progress * Give more API hints * More text updates * Hide unnecessary helpers in utils * Drop unused import and sync notebook * Save MTL updates in progress * Give more API hints * More text updates * Hide unnecessary helpers in utils * Drop unused import and sync notebook * Address comments * Rename mtl to multitask so file and tutorial match * Update Scorer import paths * Update names of loss and output funcs * Update name of ce_loss_from_outputs() * Rename SnorkelClassifier to MultitaskClassifier (#59) * Update Scorer import * Update Scorer import in vrd_tutorial * Remove unused import * [EASY] Add links to RTD in multitask tutorial (#65) * Add links to RTD in multitask tutorial * Separate sentences * Sync multitask.ipynb * Editing down code to minimal form as suggested by HE * Add Drybell tutorial (#62) * Add drybell tutorial * Add to tox and README * Install Java on Travis * Pass JAVA_HOME * Add README * Update API * Revisions to crowdsourcing tutorial (#64) * Revisions to crowdsourcing tutorial * Run tox * Address comments * Simplifying code; adding TF example; editing * Recsys novel (#61) * Recsys first commit * Backup * Added second version * Recsys modeling work * Add review processing and LFs. * First complete draft * typo * Add ipynb * Add comments, refactor * Address comments * Updated ipynb * Update tox.ini to allow sync / test for recsys (but not by default) * Address comments * Update ipynb * Address final comments * Fix determinism of TF tutorial (#67) * Fix determinism of TF tutorial * Add os PYTHONGHASHSEED back * Editing pass over text * Transfer tags w jupytext build, minor edits * Filtering abstain values * Added a stub for SFs * Trying to fix style check * Skipping env_* files in flake * Style fixes * PR changes requested by @henryre * Quieting nltk output * Separate download scripts, feedback session (#55) * Moved to getting_started * Silencing LogReg warning * Slicing spam (#18) * Forgot to sync style fix... * Add link checking (#72) * Add link checking * Fix * Only run Travis on changed tutorials (#74) * Only run Travis on changed tutorials * Fix * Address comments * Fix * Fix a couple links (#75) * Fix link * Fix links * Fix Travis branch check (#77) * Spelling fix * Stop training on dev set [EASY] (#71) * Stop training on dev set * Update image link * Add style to run envs (#78) * Add style to run envs * Simplify * Make travis faster for spouse [EASY] (#80) * Make travis faster for spouse * remove extra cell * sync * all caps for constant * More verbose build script (#81) * Add markdown build mode (#79) * Add markdown build mode * Fix kwarg * Be a bit more opinionated * [EASY] Update path to snorkel to reflect ownership transfer (#82) * Update path to snorkel to reflect ownership transfer * Restore path to snorkel-superglue on HazyResearch * Make travis only run on changed dirs (#85) * Make travis only run on changed dirs * Small fix * Add space * Update tutorials with MultitaskClassifier API changes (#68) * Update multitask and scene_graph to last_op * Update spam tutorial * Update notebooks * Sync visual_relation notebook * Update spam notebooks * Run tox -e fix * Remove unused import * Configure markdown generation (#87) * Configure markdown generation * Add comments * [EASY] Replace `mtl` with `multitask` in README (#83) * Deploy tutorial pages via Travis (#88) * Deploy tutorial pages via Travis * Fix commands * Update readme (#73) * Update readme * Address comment * Addressing comments
vincentschen
added a commit
that referenced
this pull request
Aug 14, 2019
Make travis only run on changed dirs (#85) * Make travis only run on changed dirs * Small fix * Add space Update tutorials with MultitaskClassifier API changes (#68) * Update multitask and scene_graph to last_op * Update spam tutorial * Update notebooks * Sync visual_relation notebook * Update spam notebooks * Run tox -e fix * Remove unused import Remove slice_aware param Move all SliceBinaryCLassifier to modeling Content pass Address PR comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Only does this if all changed directories are among the default envs. So if we change scripts/, then travis will check everything (which it should, as scripts may affect everything).
Test Plan
Locally changed spouse and crowd, committed, and ran
python scripts/get_tox_envs.py --plan
which showed spouse,crowdsourcing
When running with this change committed (which changes scripts/), it shows all environments.