-
Notifications
You must be signed in to change notification settings - Fork 337
Fixed #772 Add Ray Support #986
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
Merged
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
24190fa
Initial commit for adding ray support
seanlaw b037dde
Fixed missing import
seanlaw 6e4bf0a
Fixed typo
seanlaw 5aa784f
Refactored coverage reporting
seanlaw 46dbe91
Force codecov to use coverage.py report instead of generating its own
seanlaw c96e584
Specify coverage.xml in PWD
seanlaw d902e45
Added verbose flag
seanlaw 2bc2d4c
Minor change
seanlaw a900b26
Added new codecov token
seanlaw 47759e3
Added temporary break in test converage
seanlaw f177af8
Minor change
seanlaw dc76940
Changed name of coverage file
seanlaw c536ae9
Expand coverage to all tests, removed break
seanlaw ec19b48
Removed comments in workflow
seanlaw 33ca68f
Removed codecov patch/project status
seanlaw 1b620be
Minor change
seanlaw 8df0f37
Added ability to specify coverage.xml file
seanlaw 795bbef
Split xml report and displaying report
seanlaw 5a819af
Updated docstrings to include `ray` example
seanlaw 00e511a
Fixed flake8 problem
seanlaw f2f4e47
Added ray docstring examples
seanlaw a86ccb2
Fixed typo
seanlaw 1c5bb71
Reverted missing docstring
seanlaw 9154613
Minor change
seanlaw c0ca23d
Reverted docstring
seanlaw 3d93948
Merge branch 'TDAmeritrade:main' into add_ray_support
seanlaw 22e462a
Merge branch 'main' into add_ray_support
seanlaw e4940f6
Minor changes
seanlaw e33bd78
Merge branch 'main' into add_ray_support
seanlaw 3e5b1b9
Added ray.shutdown()
seanlaw 6a1c678
Changed how `step` is calculated to be more precise
seanlaw 8af81d0
Fixed missing paratheses
seanlaw File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
coverage: | ||
status: | ||
project: off | ||
patch: off |
This file contains hidden or 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
File renamed without changes.
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
#!/usr/bin/env python | ||
|
||
import requests | ||
from packaging.version import Version | ||
|
||
classifiers = ( | ||
requests.get("https://pypi.org/pypi/ray/json").json().get("info").get("classifiers") | ||
) | ||
|
||
versions = [] | ||
for c in classifiers: | ||
x = c.split() | ||
if "Python" in x: | ||
versions.append(x[-1]) | ||
|
||
versions.sort(key=Version) | ||
print(versions[-1]) | ||
NimaSarajpoor marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.