Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
96 changes: 48 additions & 48 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,48 +1,48 @@
## Issue
ISSUE > [ISSUE NAME](https://github.com/users/prathikanand7/projects/2?pane=issue&itemId=<itemid>)
## Description
Summarise what this PR does and why it is needed. Include any relevant context that a reviewer would not get from just reading the code diff.
## Type of change
<!-- Select all that apply. -->
- [ ] Bug fix
- [ ] New feature
- [ ] Refactor / cleanup
- [ ] Documentation update
- [ ] Configuration / infrastructure change
- [ ] Other: <!-- describe -->
## How has this been tested?
Describe how you verified that your changes work correctly. Include any relevant commands, test cases, or environments used. If no testing was needed, explain why.
- [ ] Unit tests
- [ ] Manual testing
- [ ] No testing required
## Notes for reviewers
Anything you want to flag for reviewers such as areas of uncertainty, known limitations, deliberate tradeoffs, or specific things you would like feedback on. Leave blank if nothing needs highlighting.
## Checklist
Go through each item before marking the PR as ready for review.
- [ ] I filled in the releated issue link
- [ ] My PR has the name of the branch
- [ ] My code follows the project's coding style
- [ ] I have reviewed my own changes before requesting review
- [ ] I have added or updated relevant documentation
- [ ] My changes do not introduce new warnings or errors
- [ ] I have not committed any secrets, credentials, or sensitive values
## Related issues
Link any issues this PR resolves or is related to.
Closes #
## Issue

ISSUE > [ISSUE NAME](https://github.com/users/prathikanand7/projects/2?pane=issue&itemId=<itemid>)

## Description

Summarise what this PR does and why it is needed. Include any relevant context that a reviewer would not get from just reading the code diff.

## Type of change

<!-- Select all that apply. -->

- [ ] Bug fix
- [ ] New feature
- [ ] Refactor / cleanup
- [ ] Documentation update
- [ ] Configuration / infrastructure change
- [ ] Other: <!-- describe -->

## How has this been tested?

Describe how you verified that your changes work correctly. Include any relevant commands, test cases, or environments used. If no testing was needed, explain why.

- [ ] Unit tests
- [ ] Manual testing
- [ ] No testing required

## Notes for reviewers

Anything you want to flag for reviewers such as areas of uncertainty, known limitations, deliberate tradeoffs, or specific things you would like feedback on. Leave blank if nothing needs highlighting.

## Checklist

Go through each item before marking the PR as ready for review.

- [ ] I filled in the releated issue link
- [ ] My PR has the name of the branch
- [ ] My code follows the project's coding style
- [ ] I have reviewed my own changes before requesting review
- [ ] I have added or updated relevant documentation
- [ ] My changes do not introduce new warnings or errors
- [ ] I have not committed any secrets, credentials, or sensitive values

## Related issues

Link any issues this PR resolves or is related to.
Closes #
2 changes: 1 addition & 1 deletion .github/workflows/e2e-notebook-deploy-and-run.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: Notebook E2E Deploy and Run
# Deploy the entire infrastructure and execute a notebook end-to-end to verify everything works together as expected.
# Deploy the entire infrastructure and execute a notebook end-to-end to verify everything works together as expected.

on:
workflow_run:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/smoke-test-worker-containerization.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: Smoke Test Worker Containerization
# This workflow builds the worker and runs a smoke test to verify that the containerization is working correctly.
# This workflow builds the worker and runs a smoke test to verify that the containerization is working correctly.

on:
push:
Expand Down Expand Up @@ -41,4 +41,4 @@ jobs:
- name: Verify Worker Containerisation
run: |
docker run --rm r-notebook-worker:test \
python -c "import os, boto3, papermill; assert os.path.exists('/app/worker.py'); print('worker-containerisation-ok')"
python -c "import os, boto3, papermill; assert os.path.exists('/app/worker.py'); print('worker-containerisation-ok')"
46 changes: 46 additions & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v5.0.0
hooks:
- id: trailing-whitespace
- id: end-of-file-fixer
- id: mixed-line-ending
args: [--fix=lf]
- id: check-merge-conflict
- id: check-json
- id: check-yaml
- id: check-added-large-files
args: [--maxkb=1500]
- id: detect-private-key
- id: detect-aws-credentials

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.11.0
hooks:
- id: ruff
args: [--fix]
files: ^(worker/.*\.py|lifewatch_batch_platform/terraform/(backend_lambdas|client_requests)/.*\.py)$
- id: ruff-format
files: ^(worker/.*\.py|lifewatch_batch_platform/terraform/(backend_lambdas|client_requests)/.*\.py)$

- repo: https://github.com/kynan/nbstripout
rev: 0.8.1
hooks:
- id: nbstripout
files: ^demo_input/.*\.ipynb$

- repo: local
hooks:
- id: terraform-fmt
name: Terraform fmt
entry: terraform fmt -recursive
language: system
pass_filenames: false
files: ^(lifewatch_batch_platform/terraform|terraform-bootstrap)/.*\.tf$

- id: frontend-eslint
name: Frontend ESLint
entry: npm --prefix frontend run lint
language: system
pass_filenames: false
files: ^frontend/(src/.*\.(ts|tsx)|package\.json|tsconfig\.json|vite\.config\.ts)$
2 changes: 1 addition & 1 deletion compress_lambdas.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ Compress-Archive -Path "$SourceDir\status.py", "$SourceDir\handle_cors.py" -Dest
Compress-Archive -Path "$SourceDir\lambda_function.py", "$SourceDir\handle_cors.py" -DestinationPath "$TargetDir\lambda.zip" -Force
Compress-Archive -Path "$SourceDir\history_list.py", "$SourceDir\handle_cors.py" -DestinationPath "$TargetDir\history_list_lambda.zip" -Force

Write-Host "Lambdas compressed to $TargetDir."
Write-Host "Lambdas compressed to $TargetDir."
2 changes: 1 addition & 1 deletion compress_lambdas.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,4 @@ zip -j "$TARGET_DIR/history_list_lambda.zip" \
"$SOURCE_DIR/history_list.py" \
"$SOURCE_DIR/handle_cors.py"

echo "Lambdas compressed to $TARGET_DIR."
echo "Lambdas compressed to $TARGET_DIR."
12 changes: 6 additions & 6 deletions demo_input/Data_cleaning.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "477a0544-ba44-4649-87a2-4448aa7bafed",
"id": "0",
"metadata": {
"tags": [
"parameters"
Expand Down Expand Up @@ -39,7 +39,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "e9a7a47a-31d5-4c61-9bae-183b1d07d7d6",
"id": "1",
"metadata": {
"vscode": {
"languageId": "r"
Expand Down Expand Up @@ -111,7 +111,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "c030c326-67b4-43d9-9213-8d58857b5d7f",
"id": "2",
"metadata": {
"vscode": {
"languageId": "r"
Expand Down Expand Up @@ -158,7 +158,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "14c595e0-33ba-4aae-890f-ad44c397429a",
"id": "3",
"metadata": {
"vscode": {
"languageId": "r"
Expand Down Expand Up @@ -260,7 +260,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "19e93b1c-73ab-44cd-9a54-4674d75a6fd9",
"id": "4",
"metadata": {
"vscode": {
"languageId": "r"
Expand Down Expand Up @@ -556,7 +556,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "b0c83712-351d-48de-88df-509c1f7e1c82",
"id": "5",
"metadata": {
"vscode": {
"languageId": "r"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "0",
"metadata": {},
"outputs": [],
"source": [
Expand All @@ -14,6 +15,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "1",
"metadata": {},
"outputs": [],
"source": [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "0",
"metadata": {},
"outputs": [],
"source": [
Expand All @@ -14,6 +15,7 @@
{
"cell_type": "code",
"execution_count": null,
"id": "1",
"metadata": {},
"outputs": [],
"source": [
Expand Down
Loading
Loading