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

dev to main #64

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
c45b943
added metadata in tag of worker image
Meetatgoogle Sep 16, 2024
bbe9065
changed to cloud logger and some auto-format changes
Meetatgoogle Sep 16, 2024
6140069
more logging changes
Meetatgoogle Sep 16, 2024
11e3e5b
dsub local changes
Meetatgoogle Sep 18, 2024
cb943d9
proposal propose action outcome JSONified
hrushikeshm-g Sep 18, 2024
aed9bce
sight.silent change and upgraded dsub
Meetatgoogle Sep 20, 2024
e8c1e45
cloud log fix
Meetatgoogle Sep 23, 2024
83a72d7
formatter changes
Meetatgoogle Oct 3, 2024
eaab9b8
more formatter changes
Meetatgoogle Oct 3, 2024
9145c19
added setuptools explicit version
abhinavprao Oct 8, 2024
8758f7c
Merge pull request #57 from theteamatx/update-setuptools-readme
bronevet-abc Oct 8, 2024
fe834e4
added instructions for formatter and setting up VM server
Meetatgoogle Oct 16, 2024
66f7365
Merge branch 'dev' of https://github.com/theteamatx/x-sight into dev
bronevet-abc Oct 16, 2024
49d97d9
message queue and testing refactoring added (#61)
hrushikeshm-g Oct 18, 2024
f0cedf6
merge-issue resovled
hrushikeshm-g Oct 18, 2024
10f85ae
deep copy replace with shallow one
hrushikeshm-g Oct 18, 2024
04990f2
updated the shallow copy
hrushikeshm-g Oct 18, 2024
f87e244
yapf pre-commit flag was missing
hrushikeshm-g Oct 18, 2024
8bb526b
Merge branch 'dev' of https://github.com/theteamatx/x-sight into dev
bronevet-abc Oct 23, 2024
7af3345
yapf and isort were conflicting
hrushikeshm-g Oct 23, 2024
0de45b8
Adding ability to track elapsed block time
bronevet-abc Oct 31, 2024
21f69bc
Final edits
bronevet-abc Nov 1, 2024
c33c50d
Service refactor, decision.py supports lists as outputs.
bronevet-abc Nov 4, 2024
7b532aa
Swap from logging.info to logging.debug
bronevet-abc Nov 4, 2024
745e7ed
Merge pull request #66 from theteamatx/service_refactor
bronevet-abc Nov 4, 2024
e583afa
Merge pull request #65 from theteamatx/block_elapsed
bronevet-abc Nov 4, 2024
3cef319
rpc changes array to map (#67)
hrushikeshm-g Nov 11, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .config/.style.yapf
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,6 @@
based_on_style = google
indent_width = 2
column_limit = 80
split_before_logical_operator = False
blank_lines_between_top_level_imports_and_variables = 1
dedent_closing_brackets = True
8 changes: 8 additions & 0 deletions .github/super-linter.env
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
VALIDATE_ALL_CODEBASE=false
VALIDATE_PYTHON_MYPY=false
VALIDATE_PYTHON_BLACK=false
VALIDATE_PYTHON_FLAKE8=false
VALIDATE_DOCKERFILE_HADOLINT=false
VALIDATE_CSS=false
DEFAULT_BRANCH='main'
FILTER_REGEX_EXCLUDE='.*\.vscode/.*\.json|.*/third_party/.*'
44 changes: 44 additions & 0 deletions .github/workflows/functional-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
name: Run test-cases

on:
push:

jobs:
test:
runs-on: ubuntu-latest

env:
PYTHONPATH: ${{ github.workspace }} # Add the tests directory to PYTHONPATH

strategy:
matrix:
test-type: [functional] # Test types

steps:
# Checkout the repository to the runner
- name: Checkout code
uses: actions/checkout@v2

- name: Print Python Path
run: |
which python
python -c "import sys; print(sys.executable)"
python -c "import sys; print(sys.path)"

- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r sight_service/requirements.txt
pip install absl-py==1.4.0
pip install colorama==0.4.6

- name: Set and print python path
run: |
echo "PYTHONPATH=$PYTHONPATH:$(pwd)/py" >> $GITHUB_ENV
which python
python -c "import sys; print(sys.executable)"
python -c "import sys; print(sys.path)"

- name: Run ${{ matrix.test-type }} test-cases
run: |
python py/tests/discover_and_run_tests.py --type ${{ matrix.test-type }}
35 changes: 26 additions & 9 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,21 +3,38 @@ repos:
rev: v4.0.1
hooks:
- id: trailing-whitespace
exclude: ^.*\.patch$
exclude: ^.*\.patch$

- id: end-of-file-fixer
exclude: ^.*\.patch$
exclude: ^.*\.patch$

- id: check-yaml

- repo: https://github.com/pre-commit/mirrors-yapf
rev: v0.32.0
# - repo: https://github.com/google/yapf
# rev: v0.31.0
# hooks:
# - id: yapf
# args: ["-ir", "--style", ".config/.style.yapf"]

# - repo: https://github.com/pre-commit/mirrors-isort
# rev: v5.10.1
# hooks:
# - id: isort
# args: ["--settings-path", ".config/.isort.cfg"]

- repo: https://github.com/google/yapf
rev: v0.31.0
hooks:
- id: yapf
args: ['--style', '.config/.style.yapf']
name: yapf and isort
entry: bash -c "yapf --style .config/.style.yapf -i $@ && isort --settings-path .config/.isort.cfg $@"
language: system
types: [python]

- repo: https://github.com/pre-commit/mirrors-isort
rev: v5.10.1
- repo: local
hooks:
- id: isort
args: ['--settings-path', '.config/.isort.cfg']
- id: run-functional-tests
name: Run all functional unit-tests
entry: python py/tests/discover_and_run_tests.py --type functional
language: system
always_run: true
Loading
Loading