Skip to content

Autofeed v2 #22

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

Merged
merged 8 commits into from
Mar 10, 2022
Merged

Autofeed v2 #22

merged 8 commits into from
Mar 10, 2022

Conversation

pbailie
Copy link
Contributor

@pbailie pbailie commented Nov 17, 2021

The autofeed script has been refactored and improved.

  • More maintainable "upsert" queries now use on conflict (...) do update. Original code was written for Postgresql 9.4 which did not support on conflict clauses.
  • Code has been better separated into a few different class files to make it easier to maintain these pieces.
  • Upsert processing is compartmentalized for each course. That is, a data anomaly for one course will prevent that course from being upserted (to preserve data integrity), but other courses will be upserted normally provided they do not have data anomalies, as well.

Student enrollments report script

  • This is run once before the autofeed and once after.
  • The first run reads the database and caches enrollment values to a temp file.
  • The second run reads the temp file and compares the values with the database. A report is created showing student enrollment adds and drops.
  • The report shows the number of students who add or drops from courses registered in Submitty. Only numbers. No names.
  • The report may be optionally emailed.

Now checks dropped student ratio and blocks upsert if the ratio is beyond a cutoff value set in config.php
Fix crash bug when there are no mapped courses.
Can now run properly from any directory using full execution path.
@bmcutler bmcutler merged commit 0071454 into master Mar 10, 2022
@bmcutler bmcutler deleted the autofeed-v2 branch March 10, 2022 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants