Skip to content

Conversation

@its-sushant
Copy link
Contributor

@its-sushant its-sushant commented Jun 24, 2022

Description

Added a agent for logistic regression model.

Files

  • agents/logisticRegression.py (Python file to run logreg model)

How to use

  • Running logisticRegression agent
    atarashi -a logisticRegression /path/to/file.c

  • Checking accuracy using evaluator.py
    python3 path/to/evaluator.py -a logisticRegression

Accuracy

The accuracy score that has been tested using evaluator.py is 63%

Screenshot from 2022-06-23 19-03-59

CC: @Kaushl2208 @GMishx @hastagAB @ag4ums @vasudevmaduri

@its-sushant its-sushant force-pushed the minerva-test branch 2 times, most recently from 3c5b9d1 to dd9577f Compare June 28, 2022 10:42
Copy link
Member

@Kaushl2208 Kaushl2208 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, Needs testing :)

Will try and test it by weekend :)

@Kaushl2208
Copy link
Member

Hi @its-sushant ,
Upon reviewing the recent PRs, it appears that these are extensions to PR #69. Specifically, PRs #100 and #102 serve as actual extensions, while PR #101 adds new agents/models to the list.

Given the overlap and interdependencies, it seems more practical to consolidate these changes into a single PR. This approach could minimize the risk of merge conflicts arising from common modifications and streamline the testing process.

Could you consider merging these updates into one PR to enhance efficiency and coherence in our workflow?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants