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

Adds Pinset support #38

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Adds Pinset support #38

wants to merge 3 commits into from

Conversation

alfonsodelavega
Copy link
Member

This branch updates the playground to support Pinset:

  • Includes a basic PSL example Generate Project CSVs
  • Generates one CSV for each dataset rule (two in the example)
  • Tested locally with the changes PRed to the backend
  • Includes Cypress tests (caveat: in my current machine tests only run in chromium, for some reason Cypress cannot find firefox when running)

Possible improvements:

  • Right now no language uses two output panels, and no code output has the option to be prettyfied. Maybe adding a formatted table that pretty-prints each CSV could be nice (or a button to "render" the CSV like it's done with the model diagrams). I opted for showing directly the CSV contents in case the user wanted to copy the results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant