Skip to content

Conversation

@tupui
Copy link
Member

@tupui tupui commented Oct 31, 2023

This is a POC of a sampling app.

Screenshot 2023-10-31 at 23 21 11

By having multiple apps, we can either have an index (need to work on the html) or leave this out and reference the apps ourselves. Here is an example of such index.

Screenshot 2023-10-31 at 23 19 52

cc @gnopik

@tupui tupui added enhancement New feature or request dashboard labels Oct 31, 2023
@tupui
Copy link
Member Author

tupui commented Oct 31, 2023

Something to solve/mitigate. Having a dynamic list for the parameters is tricky to update automagically. For the moment, ones need to change the number of samples or the method after having changed a parametrisation. Non ideal, but it works for now without magic in the code.

Another thing to watch out, for the parametrisation of the distributions: we have to allow passing a dictionary. If this really goes into production we will have to look into hardening this as user can input anything and we actually execute this on the backend. We only need to allow passing a dictionary with limited support to str, int, float.

@gnopik
Copy link
Contributor

gnopik commented Nov 1, 2023

Whichever would be easier perhaps?

My only suggestion is to make it dumb-proof by calling

  • the original dashboard: SimDec dashboard
  • Sampling app: Data generation (sampling)

@tupui tupui marked this pull request as ready for review February 16, 2024 21:12
Copy link
Member Author

@tupui tupui left a comment

Choose a reason for hiding this comment

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

Merging this now to see how it goes and get first round of feedbacks 😃 (And crossing fingers it deploys without issue hum)

@tupui tupui merged commit a23777f into main Feb 16, 2024
@tupui tupui deleted the sampling_app branch February 16, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants