Skip to content

Conversation

jmaupetit
Copy link
Contributor

@jmaupetit jmaupetit commented Jan 26, 2023

⚠️ This PR is a POC and should not be merged ⚠️

Purpose

Apache superset is an alternative to grafana for dashboarding. It's worth testing it for learning analytics.

Proposal

  • add xAPI statements as a dataset (Elasticsearch)
  • integrate row level security macro (courses filtering)

References

OpenEdx plan to integrate Superset as the visualization tool for learning traces. For reference, please read:

Apache superset is an alternative to grafana for dashboarding. It's
worth testing it for learning analytics.
@jmaupetit jmaupetit self-assigned this Jan 26, 2023
@jmaupetit jmaupetit assigned wilbrdt and unassigned wilbrdt Jan 26, 2023
@jmaupetit jmaupetit requested a review from wilbrdt January 26, 2023 14:57
Copy link
Contributor

@quitterie-lcs quitterie-lcs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixtures \
hooks
hooks \
init-superset
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it necessary to include superset in the bootstrap? As it is experimental for now ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll write a disclaimer in this PR. This work is not intended to be merged, I opened this PR to share my investigations and let people test this tool. 😉

zcat ./fixtures/elasticsearch/lrs.json.gz | \
$(COMPOSE_RUN) patch_statements_date | \
$(COMPOSE_RUN) -T ralph push -b es --es-index statements-fixtures && \
$(COMPOSE_RUN) -T ralph push -b es && \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this index ?

Comment on lines +50 to +51
1. `docker-compose down -v`
2. `docker-compose up`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
1. `docker-compose down -v`
2. `docker-compose up`
1. `docker compose down -v`
2. `docker compose up`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants