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

Pull all reports from Insights #31

Closed
Tracked by #11
bmtcril opened this issue Jan 13, 2023 · 6 comments
Closed
Tracked by #11

Pull all reports from Insights #31

bmtcril opened this issue Jan 13, 2023 · 6 comments
Assignees
Labels
good first issue A good task for a newcomer to start with help wanted Ready to be picked up by anyone in the community

Comments

@bmtcril
Copy link
Contributor

bmtcril commented Jan 13, 2023

To help build the list of reports needed for OARS we need to go through the Insights code and organize a list of the reports.

Reports in the dashboard should all be listed here: https://github.com/openedx/edx-analytics-dashboard/blob/master/analytics_dashboard/courses/urls.py

Most upstream pipeline code that generates the backing tables is here: https://github.com/openedx/edx-analytics-pipeline

Reports should be labeled into buckets:

  • Can be done in OARS v1 with existing xAPI events
  • Can be done in OARS v1 with new xAPI events
  • Cannot be done in OARS v1

And a level of effort for ones that can be done with existing events:

  • Low - can be done in less than a day
  • Medium - can be done in one-two days
  • High - can be done in a week
  • Epic - will likely take more than a week

AC:

  • A spreadsheet with all existing Insights events with columns for:
    • Links to their locations in GitHub (in dashboard and any upstream pipeline jobs)
    • The above fields for difficulty and ability to be done with existing events
    • Is the report documented and on by default, or is it one of the "too slow to be run by default" / hidden ones
    • What data sources and tracking event types does this report use?
    • Any notes that may be of interest to the implementer
@bmtcril bmtcril mentioned this issue Jan 13, 2023
2 tasks
@bmtcril bmtcril added good first issue A good task for a newcomer to start with help wanted Ready to be picked up by anyone in the community labels Jan 13, 2023
@Ian2012
Copy link
Contributor

Ian2012 commented Apr 10, 2023

Here is a template with the report list https://docs.google.com/spreadsheets/d/1PpnzmJDuZ69P8NMzDCDkwnxyIgwnvmKpIxFnjI-LW7U/edit#gid=0 (WIP)

For the column "Link in any upstream pipeline jobs" should I link the reducer code?

@bmtcril
Copy link
Contributor Author

bmtcril commented Apr 11, 2023

This is looking great, thanks for taking on the work! Yes, I was hoping to link back to relevant reducer code there so we can learn from any adjustments that may be present there (if they're relevant).

If you're taking the whole task on, you should be able to comment assign me on this ticket to get it assigned to you. It's done via GH action, so let me know if you run into any issues. Then if you could please update the status to doing that would be great!

@Ian2012
Copy link
Contributor

Ian2012 commented Apr 11, 2023

assign me

@bmtcril
Copy link
Contributor Author

bmtcril commented Apr 20, 2023

@Ian2012 are you still doing this task?

@Ian2012
Copy link
Contributor

Ian2012 commented Apr 20, 2023

Not currently @bmtcril

@bmtcril
Copy link
Contributor Author

bmtcril commented Apr 21, 2023

Thanks @Ian2012 , I think I've got it as far as it needs to go. Details on the calculations are included in the Insights docs and we won't have anything like the source data they had, so chasing down all of the links to code is probably not worthwhile. I'm going to call this one done!

@bmtcril bmtcril closed this as completed Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue A good task for a newcomer to start with help wanted Ready to be picked up by anyone in the community
Projects
Development

No branches or pull requests

2 participants