Skip to content

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

Open
jmaupetit wants to merge 1 commit intomainfrom
poc-superset
Open

⚗️(project) add draft Apache Superset integration#65
jmaupetit wants to merge 1 commit intomainfrom
poc-superset

Conversation

@jmaupetit
Copy link
Copy Markdown
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
Copy Markdown
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.

Comment thread Makefile
fixtures \
hooks
hooks \
init-superset
Copy link
Copy Markdown
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
Copy Markdown
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. 😉

Comment thread Makefile
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
Copy Markdown
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 thread superset/README.md
Comment on lines +50 to +51
1. `docker-compose down -v`
2. `docker-compose up`
Copy link
Copy Markdown
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