-
Notifications
You must be signed in to change notification settings - Fork 4
UV-10 -Update algo for cocos #11
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
Conversation
6d70e2e
to
f98ff46
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please update readme with instructions for where to place datasets once downloaded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a requirements file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix typos in readme file, also include step by step actions to take when testing with cocos
requirements.txt
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
limit to minimum requirements required for file
numpy==2.0.0
pandas==2.2.2
scikit-learn==1.5.1
torch==2.3.1
joblib==1.4.2
matplotlib==3.9.1
rul-turbofan/requirements.txt
Outdated
torch==2.3.1 | ||
joblib==1.4.2 | ||
matplotlib==3.9.1 | ||
seaborn==0.13.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add an empty line at the end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FilipCivljak Please rebase and resolve comments. |
8d99634
to
7d3fc90
Compare
Dockerfile for Fraud Detection Renamed Dockerfiles, modified rul and covid prediction scripts, and added new Dockerfiles Renamed inference Dockerfiles to Dockerfile.infer Change command in Dockerfile from python3 to python Changes on requiremnts.txt and python.yaml
7d3fc90
to
e861ffe
Compare
Signed-off-by: filip <filipcivljak007@gmail.com>
@SammyOina Please review. |
@SammyOina please review |
1 similar comment
@SammyOina please review |
Signed-off-by: filip <filipcivljak007@gmail.com>
.github/workflows/python.yaml
Outdated
@@ -7,13 +7,15 @@ on: | |||
paths: | |||
- "covid19/**" | |||
- "fraud-detection/**" | |||
- "rul-turbofan"/**" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "rul-turbofan"/**" | |
- "rul-turbofan/**" |
.github/workflows/python.yaml
Outdated
- ".github/**" | ||
pull_request: | ||
branches: | ||
- main | ||
paths: | ||
- "covid19/**" | ||
- "fraud-detection/**" | ||
- "rul-turbofan"/**" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- "rul-turbofan"/**" | |
- "rul-turbofan/**" |
Signed-off-by: filip <filipcivljak007@gmail.com>
rul-turbofan/README.md
Outdated
``` | ||
|
||
```bash | ||
unzip results.zip -d results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Close the code block
|
||
```bash | ||
go run ./cmd/cli/main.go checksum ../ai/rul-turbofan/datasets/test_FD001.txt | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add new line after the code block
Signed-off-by: filip <filipcivljak007@gmail.com>
resolves: #10