Skip to content

Commit

Permalink
Vaishaal/tokenize shuffle generator (#85)
Browse files Browse the repository at this point in the history
* tokenize shuffle fixes

* tokenize shuffle fixes

* tokenize shuffle fixes specify tmp dir

* tokenize shuffle bug fixes, remove counter from inner loop

* make batch_format=pandas remove extra loop

* added some rudimentary tests to tokenize shuffle

* added tests that actually pass

* change ray version

* grab dummy credentials for aws stuff to work

* change ci version

* ci version update

* wget dummy creds

* wget ci fixes

* add not s3 to make test

* fix pytest -m command
  • Loading branch information
Vaishaal authored Nov 22, 2023
1 parent af463ec commit 7fc5cb1
Show file tree
Hide file tree
Showing 6 changed files with 289 additions and 198 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
- name: Set up Python 3.8
uses: actions/setup-python@v2
with:
python-version: 3.8
python-version: "3.10"
- name: Install
run: |
sudo apt-get update
Expand All @@ -32,7 +32,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [3.8]
python-version: ["3.10"]

steps:
- uses: actions/checkout@v2
Expand All @@ -42,6 +42,8 @@ jobs:
python-version: ${{ matrix.python-version }}
- name: Install
run: |
mkdir $HOME/.aws/
wget https://gist.githubusercontent.com/Vaishaal/f109bfab6a194a93040ae2a19b6be251/raw/7d8026ae234d77ba1ca29b1f9d114c6780308ae4/dummy_creds -O $HOME/.aws/credentials
sudo apt-get update
python3 -m venv .env
source .env/bin/activate
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ black: ## [Local development] Auto-format python code using black

TEST_ARGS = tests ## set default to run all tests
test: ## [Local development] Run unit tests
python -m pytest -x -s -v $(TEST_ARGS) -m "not gpu"
python -m pytest -x -s -v $(TEST_ARGS) -m "not gpu and not s3"

test-gpu: ## [Local development] Run unit tests
python -m pytest -x -s -v $(TEST_ARGS) -m gpu
Expand Down
Loading

0 comments on commit 7fc5cb1

Please sign in to comment.