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

[Feature] setup pytest on existing samples #124

Merged
merged 8 commits into from
Feb 12, 2023

Conversation

Udayraj123
Copy link
Owner

@Udayraj123 Udayraj123 commented Jan 21, 2023

Setup Pytest using the samples. Integrate it in pre-commit and pre-push git hooks
PR#2 of a chained PR.
Prev Next

@Udayraj123 Udayraj123 changed the title Feature/pytest [Feature] setup pytest on existing samples Jan 22, 2023
@Udayraj123 Udayraj123 marked this pull request as ready for review January 24, 2023 06:47
* fix: update snapshots

feat: simplify template schema and block logic; consume it; use fields terminology; update all samples

fix: bug fixes

* fix: refactor template.py; minor template json cleanup

* fix: changes after updating wiki

* feat: setup tests for template validations; fixed few bugs

fix: renaming

fix: test

* fix: refactor

fix: refactor tests structure and add utils

* fix: minor fixes

* fix: reorder imports; rename question -> field

* fix: review changes

* fix: remove streak logic
@Udayraj123 Udayraj123 merged commit 426b7a6 into feature/evalution-refactor Feb 12, 2023
@Udayraj123 Udayraj123 deleted the feature/pytest branch February 12, 2023 06:56
Udayraj123 added a commit that referenced this pull request Feb 12, 2023
* fix: add upsc mock; refactor; create evaluation-schema

fix: refactor; add sample evaluation.json

fix: update gitignore and rename ignored folder

fix: setup skeleton; refactor

* fix: refactor; change defaults to .py; pickup config.json recursively; wip pass instance

* fix: update evaluation-schema and sample json wip

fix: refactor; connect evaluation schema

fix: checked validation using evaluation schema

* refactor: separate static vs instance ImageUtils; breakfown files further; move constants/configs to better places;

fix: refactor; move instance ops to core; minor changes

refactor: extract tuning config from instance

fix: add screeninfo for simple window info

* fix: load json error handling; dynamic config json working for sample5

* fix: use formatted strings at all places necessary

* fix: correct config schema

* fix: working evaluation code

fix: evaluation schema fixes

* fix: add range operator support; add validations for marking scheme and omr response

* fix: add a sample with answer_key.csv

* fix: minor fixes

* fix: foundation for answer_key_image_path; refactoring; minor changes

* fix: remove unnecessary comments; fix minor comments

* fix: further on foundation for csv_from_image feature; refactoring

* fix: stronger validations; conditionality in streak scheme; joinpath refactor

* fix: support multi-correct and basic multi-weighted; update explanations

* fix: csv load custom key support

fix: additional warning

* fix: add evaluation configs in some samples

* fix: load answer key from image with working samples

fix: compress images

* fix: compress images

* fix: update samples; add readme for template shift sample

* fix: reduce images; minor changes

* [Feature] setup pytest on existing samples (#124)

* fix: load answer key from image with working samples

* fix: pytests set up with snapshots for all samples

* fix: run sample1 on pre-commit; run all tests on pre-push

fix: add default_install_hook_types

fix: add explicit stages

* fix: update snapshots

* fix: bug fixes

* [Feature] Simplify template jsons (#127)

* fix: update snapshots

feat: simplify template schema and block logic; consume it; use fields terminology; update all samples

fix: bug fixes

* fix: refactor template.py; minor template json cleanup

* fix: changes after updating wiki

* feat: setup tests for template validations; fixed few bugs

fix: renaming

fix: test

* fix: refactor

fix: refactor tests structure and add utils

* fix: minor fixes

* fix: reorder imports; rename question -> field

* fix: review changes

* fix: remove streak logic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant