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

Update the audit jobs for stats model #7100

Open
ankita240796 opened this issue Jul 5, 2019 · 19 comments
Open

Update the audit jobs for stats model #7100

ankita240796 opened this issue Jul 5, 2019 · 19 comments
Labels

Comments

@ankita240796
Copy link
Contributor

Update audit jobs for stats models and move them from core/domain/stats_jobs_one_off.py to core/domain/prod_validation_jobs_one_off.py. Once the update is done, the tests created in #7099 should also be updated by removing the statistics models from the list of excluded models.

@jacobdavis11
Copy link
Member

@ankita240796 how is this going?

@brianrodri
Copy link
Contributor

Hi @ankita240796, going to unassign you for now so someone else can attempt it, feel free to take over again if you're still interested.

@ankita240796
Copy link
Contributor Author

Sure @brianrodri, Thanks!

@mridullpandey
Copy link

hi @ankita240796 may i take this up...!

@mridullpandey
Copy link

i'm working on this:-)

@ankita240796
Copy link
Contributor Author

Sure @mridullpandey, Go for it! Do let me know if you need any help! Thanks!

@mridullpandey
Copy link

thanks @ankita240796

@ghost
Copy link

ghost commented Nov 12, 2019

is there any progress on this issue?

@ankita240796 ankita240796 removed their assignment Apr 3, 2020
@aggarwalpulkit596
Copy link
Contributor

Can i take this one @ankita240796 @seanlip

@Arnesh07
Copy link
Contributor

Arnesh07 commented Apr 7, 2020

I have assigned you! @aggarwalpulkit596
Thanks!

@aggarwalpulkit596
Copy link
Contributor

@seanlip @ankita240796 is this issue still open?

@ankita240796
Copy link
Contributor Author

Yes, you can take this up @aggarwalpulkit596. Thanks!

@aggarwalpulkit596
Copy link
Contributor

@ankita240796 can you explain this which stats model are you taking about

@ankita240796
Copy link
Contributor Author

@aggarwalpulkit596
Copy link
Contributor

Thanks I'll take a look

@BenHenning
Copy link
Sponsor Member

@aggarwalpulkit596 are you still working on this issue?

@aggarwalpulkit596
Copy link
Contributor

No 😅

@BenHenning
Copy link
Sponsor Member

Ack, de-assigning.

@BenHenning
Copy link
Sponsor Member

BenHenning commented Jun 1, 2020

@MegrezZhu suggest starting on this issue.

seanlip pushed a commit that referenced this issue Jul 8, 2020
* Update audit job for playthrough model

* Split validator functions

* Remove unused code

* Fix linting & review change

* Update core/domain/prod_validation_jobs_one_off.py

Co-authored-by: Brian Rodriguez <brian@brianrodri.com>

* Add requested changes

* Change constant name

* Add debug information in validation messages

* Remove unused constant

* Remove referece_count

* Fix test errors after merging develop

Co-authored-by: Brian Rodriguez <brian@brianrodri.com>
shavavo pushed a commit to shavavo/oppia that referenced this issue Aug 20, 2020
…#9546)

* Update audit job for playthrough model

* Split validator functions

* Remove unused code

* Fix linting & review change

* Update core/domain/prod_validation_jobs_one_off.py

Co-authored-by: Brian Rodriguez <brian@brianrodri.com>

* Add requested changes

* Change constant name

* Add debug information in validation messages

* Remove unused constant

* Remove referece_count

* Fix test errors after merging develop

Co-authored-by: Brian Rodriguez <brian@brianrodri.com>
@seanlip seanlip removed this from QA/Testing in Lesson analytics [INACTIVE] Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests