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

⚗️(project) add draft Apache Superset integration #65

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

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.

@@ -50,7 +50,8 @@ bootstrap: \
build \
compile \
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. 😉

@@ -86,7 +87,7 @@ fixtures: ## Load test data (for development)
@$(WAIT_MYSQL)
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