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.

Sign up for GitHub

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

Jump to bottom

Implement the basic backend with API #14

Merged
merged 15 commits into from
Oct 14, 2024
Merged

Implement the basic backend with API #14

merged 15 commits into from
Oct 14, 2024

Conversation

Iamhexi
Copy link
Owner

@Iamhexi Iamhexi commented Oct 8, 2024

Implemented configuration via config.yaml (or any other file passed via --config <path/to/configuration.yaml> flag) and made possible to run backend.

@Iamhexi Iamhexi added the feature New feature or request label Oct 8, 2024
@Iamhexi Iamhexi self-assigned this Oct 8, 2024
Lukasz825700516
Lukasz825700516 reviewed Oct 10, 2024
View reviewed changes
Copy link

@Lukasz825700516 Lukasz825700516 left a comment

Choose a reason for hiding this comment

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

Get uwu'ed

knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
poetry.lock Outdated Show resolved Hide resolved
knowledge_verificator/utils/configuration_parser.py Outdated Show resolved Hide resolved
knowledge_verificator/utils/configuration_parser.py Outdated Show resolved Hide resolved
knowledge_verificator/utils/configuration_parser.py Outdated Show resolved Hide resolved
knowledge_verificator/utils/configuration_parser.py Outdated Show resolved Hide resolved
@Lukasz825700516
Copy link

Just took a glance on the code, I haven't attempted to lint it or run and look how it runs.

Iamhexi
Iamhexi commented Oct 10, 2024
View reviewed changes
Copy link
Owner Author

@Iamhexi Iamhexi left a comment

Choose a reason for hiding this comment

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

I am also pushing changes so you may re-review them.

knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/command_line.py Show resolved Hide resolved
knowledge_verificator/command_line.py Outdated Show resolved Hide resolved
knowledge_verificator/command_line.py Show resolved Hide resolved
knowledge_verificator/command_line.py Show resolved Hide resolved
knowledge_verificator/command_line.py Show resolved Hide resolved
@Iamhexi Iamhexi marked this pull request as ready for review October 10, 2024 15:42
Lukasz825700516
Lukasz825700516 reviewed Oct 10, 2024
View reviewed changes
Copy link

@Lukasz825700516 Lukasz825700516 left a comment

Choose a reason for hiding this comment

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

 _____
< owo >
 -----
        \   ^__^
         \  (oo)\_______
            (__)\       )\/\
                ||----w |
                ||     ||

knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/backend.py Outdated Show resolved Hide resolved
knowledge_verificator/command_line.py Show resolved Hide resolved
knowledge_verificator/materials.py Outdated Show resolved Hide resolved
knowledge_verificator/materials.py Outdated Show resolved Hide resolved
knowledge_verificator/materials.py Outdated Show resolved Hide resolved
knowledge_verificator/materials.py Show resolved Hide resolved
Lukasz825700516
Lukasz825700516 reviewed Oct 11, 2024
View reviewed changes
Copy link

@Lukasz825700516 Lukasz825700516 left a comment

Choose a reason for hiding this comment

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

 ______________
< looks better >
 --------------
                       \                    ^    /^
                        \                  / \  // \
                         \   |\___/|      /   \//  .\
                          \  /O  O  \__  /    //  | \ \           *----*
                            /     /  \/_/    //   |  \  \          \   |
                            @___@`    \/_   //    |   \   \         \/\ \
                           0/0/|       \/_ //     |    \    \         \  \
                       0/0/0/0/|        \///      |     \     \       |  |
                    0/0/0/0/0/_|_ /   (  //       |      \     _\     |  /
                 0/0/0/0/0/0/`/,_ _ _/  ) ; -.    |    _ _\.-~       /   /
                             ,-}        _      *-.|.-~-.           .~    ~
            \     \__/        `/\      /                 ~-. _ .-~      /
             \____(oo)           *.   }            {                   /
             (    (--)          .----~-.\        \-`                 .~
             //__\\  \__ Ack!   ///.----..<        \             _ -~
            //    \\               ///-._ _ _ _ _ _ _{^ - - - - ~

knowledge_verificator/answer_chooser.py Outdated Show resolved Hide resolved
@Iamhexi Iamhexi merged commit e555d2d into main Oct 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lukasz825700516 Lukasz825700516 Lukasz825700516 left review comments

Assignees

@Iamhexi Iamhexi

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Iamhexi @Lukasz825700516