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

Verbose backend failure responses #154

Merged
merged 39 commits into from
Mar 8, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
8e90eb1
WIP usage of custom error class to discern between user error and sys…
budak7273 Feb 1, 2023
2df6afc
Merge branch 'main' into verbose-backend-failure-responses
budak7273 Feb 7, 2023
754442a
Merge branch 'main' into verbose-backend-failure-responses
SpookyBeverage Feb 9, 2023
1a0bc1f
Move old team's tests into another folder
budak7273 Feb 10, 2023
87e020f
Move plots code to its own folder
budak7273 Feb 11, 2023
6b468c9
Move config related stuff to its own file
budak7273 Feb 11, 2023
6340b79
Breaking into functions and using nice error messages progress
budak7273 Feb 11, 2023
9ecae87
Breaking into functions and using nice error messages progress 2
budak7273 Feb 11, 2023
9119060
Don't always restart the mongodb container when the host's docker res…
budak7273 Feb 11, 2023
d0cfa63
Also install python dev dependencies on the host machine
budak7273 Feb 11, 2023
3f4244b
Extract constant
budak7273 Feb 11, 2023
30d7d19
Use our custom errors in plot generation
budak7273 Feb 11, 2023
930f83d
Start linter in the right root directory (not sure why this only crop…
budak7273 Feb 11, 2023
1a6ad79
Added 2 new test files
SpookyBeverage Feb 11, 2023
faf3751
Fix imports (kind of concerning that the linter didn't pick these up)
budak7273 Feb 11, 2023
3da1f9f
oops typo!
SpookyBeverage Feb 11, 2023
e1d7fea
Merge branch 'verbose-backend-failure-responses' of https://github.co…
SpookyBeverage Feb 11, 2023
99c51a8
no =
SpookyBeverage Feb 11, 2023
5fa04c5
Merge branch 'main' into verbose-backend-failure-responses
budak7273 Feb 11, 2023
d5ba358
DumbTextArea placeholder text
budak7273 Feb 11, 2023
9f3d572
Example inputs for addNumsWithConstants
budak7273 Feb 11, 2023
e0dc3c5
Organize our example experiments. Make addNumsTimed have a parameteri…
budak7273 Feb 11, 2023
3f40c3f
Move experiment running code to its own file
budak7273 Feb 12, 2023
2d352e1
Fix python auto-import and linter settings
budak7273 Feb 12, 2023
fce98fa
Add parameters and explanations of the purpose of each example experi…
budak7273 Feb 12, 2023
91d5515
Rename the devcontainer file for now so vscode stops suggesting peopl…
budak7273 Feb 12, 2023
deb6922
Adding Some more info on how to test
SpookyBeverage Feb 12, 2023
798a326
Merge branch 'verbose-backend-failure-responses' of https://github.co…
SpookyBeverage Feb 12, 2023
7ff36df
Fix broken import
budak7273 Feb 15, 2023
76b6259
Added file logs
SpookyBeverage Mar 6, 2023
bc6f27a
Merge pull request #152 from AutomatingSciencePipeline/Sotring-Logs
rrobinson281 Mar 7, 2023
b32b012
Update completion message to reference the correct venv name (backend…
budak7273 Mar 8, 2023
d712aea
Switch to backslash paths so vscode stops randomly getting confused a…
budak7273 Mar 8, 2023
0b21827
Docs update
budak7273 Mar 8, 2023
da564db
Add extension for previewing zip file contents (so we can see inside …
budak7273 Mar 8, 2023
c05a094
Specify full path to venv activate
budak7273 Mar 8, 2023
8e7a606
Top level exception catch that actually logs what went wrong so we ca…
budak7273 Mar 8, 2023
691164f
Handle the case of typos in the File Output or Result Output lines
budak7273 Mar 8, 2023
eb0267f
Merge branch 'main' into verbose-backend-failure-responses
budak7273 Mar 8, 2023
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
1 change: 1 addition & 0 deletions .vscode/extensions.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
"ms-python.pylint",
"ms-python.vscode-pylance",
"bungcip.better-toml", // Pyenv pipfile is toml
"slevesque.vscode-zipexplorer", // see inside experiment output zip files via right click -> explore

// Setup Scripts
"timonwong.shellcheck",
Expand Down
24 changes: 18 additions & 6 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"headlessui",
"hyperparameters",
"hyperparams",
"jsonify",
"mantine",
"matplotlib",
"mouck",
Expand Down Expand Up @@ -45,16 +46,27 @@

// Tell the Python vscode extension to look at the venv's python install before any found on the system
// https://github.com/microsoft/vscode-python/wiki/Setting-descriptions#pythondefaultinterpreterpath
"python.defaultInterpreterPath": "./apps/backend/.venv/Scripts/python.exe",
"python.defaultInterpreterPath": ".\\apps\\backend\\.venv\\Scripts\\python.exe",
// Tell the Pylint extension to look for its rules file off of this folder instead of the repo root
"python.linting.cwd": "${workspaceFolder}/apps/backend",
// Tell the Python extension to also follow imports from inside the backend directory, so that the auto-suggested imports don't drill down from the repo root folder
"python.analysis.extraPaths": [
"./apps/backend"
],
"python.analysis.typeCheckingMode": "basic",
"python.formatting.provider": "yapf",
// Default line length settings often causes it to split logging messages over multiple lines
"python.formatting.yapfArgs": ["--style", "{based_on_style: pep8, column_limit: 999}"],
"python.linting.lintOnSave": true,

// From the past team (unclear if they are still needed)
"python.testing.pytestArgs": [
"apps"
],
"python.testing.unittestEnabled": false,
"python.testing.pytestEnabled": true,
"python.formatting.provider": "yapf",
// Default line length settings often causes it to split logging messages over multiple lines
"python.formatting.yapfArgs": ["--style", "{based_on_style: pep8, column_limit: 999}"],
"python.linting.lintOnSave": true
"python.testing.pytestEnabled": true

// ======================= Frontend =============================

// Below this point is uncategorized (please categorize)
}
2 changes: 1 addition & 1 deletion Monorepo.wiki
1 change: 1 addition & 0 deletions apps/backend/.pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,5 @@ disable=
C0301, # line too long
C0114, # missing-module-docstring
C0116, # missing-function-docstring
C0115, # missing-class-docstring
C0103, # we chose to use camel case to be consistent with our frontend code
Loading