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

Add ancestor information to summary and session log exports #12691

Closed
rtibbles opened this issue Oct 1, 2024 · 2 comments · Fixed by #12782
Closed

Add ancestor information to summary and session log exports #12691

rtibbles opened this issue Oct 1, 2024 · 2 comments · Fixed by #12782
Assignees
Labels
APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) DEV: backend Python, databases, networking, filesystem... P1 - important Priority: High impact on UX

Comments

@rtibbles
Copy link
Member

rtibbles commented Oct 1, 2024

Observed behavior

Some users have asked for additional information showing the topic hierarchy for each resource node in the export logs. We had prototyped this previously, but it caused huge slowdowns during the export process.

However, now that the ancestor information is directly annotated onto the content nodes, we can reuse this for quick access to this information.

Expected behavior

The ancestors information should be unpacked and the longest ancestor chain used to determine how many topic headers to include. Then each ancestor for each row should be put into a topic header until the ancestors are exhausted.

The topic headers should be a string in the form "Topic level {n}" starting at 1.

User-facing consequences

This should hopefully allow users who have made deliberate choices in their topic hierarchies to use that information in their data analysis.

@rtibbles rtibbles added P1 - important Priority: High impact on UX APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) labels Oct 1, 2024
@rtibbles rtibbles added the DEV: backend Python, databases, networking, filesystem... label Oct 1, 2024
@thesujai
Copy link
Contributor

Can i give it a shot?

@nucleogenesis
Copy link
Member

@thesujai in the kolibri.core.logger.management.commands module you'll find the exportlogs.py file which is where the root of the work should be, I think - but here in the kolibri.core.logger module, the csv_export file appears to be where the document is generated.

To test this, you can go to the Facility -> Data section while signed in as an admin, there you can click to generate the logs.

This all starts from around here I think (just for context, as the work should only involve the backend module(s) linked above).

If you have any questions reach out on Slack or here! Thanks again for taking this one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Facility Re: Facility App (user/class management, facility settings, csv import/export, etc.) DEV: backend Python, databases, networking, filesystem... P1 - important Priority: High impact on UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants