Skip to content

Layout for playground concept graph#67

Merged
mmaelicke merged 1 commit into
mainfrom
uncertainty_draft
Apr 22, 2022
Merged

Layout for playground concept graph#67
mmaelicke merged 1 commit into
mainfrom
uncertainty_draft

Conversation

@mmaelicke
Copy link
Copy Markdown
Member

@cojacoo,

I quckly drafted the concept graph figure for learning about Knightian uncertainty. I basically rebuilts what I have seen on your presentation. It is meant to be the starting point into the actual application to explain the concept.

You can run that app by checking out the branch and locally install the ruins package

git checkout uncertaintry_draft
pip install -e .

And then run within the repo

streamlit run ruins/apps/uncertainty.py

Is the general layout and flow through this example what you had in mind?
Obviously, the labels and text need some replacements and a little bit of work is needed for the figure, but this draft should point into the right direction.

@mmaelicke mmaelicke added the uncertainty explorer Anything related to the uncertainty app label Apr 22, 2022
@mmaelicke mmaelicke added this to the APP: Uncertainty milestone Apr 22, 2022
@mmaelicke mmaelicke requested a review from cojacoo April 22, 2022 07:24
@mmaelicke mmaelicke self-assigned this Apr 22, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2022

Codecov Report

Merging #67 (c543755) into main (06ad9d7) will decrease coverage by 1.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   20.09%   19.09%   -1.01%     
==========================================
  Files          19       19              
  Lines        1612     1697      +85     
==========================================
  Hits          324      324              
- Misses       1288     1373      +85     
Impacted Files Coverage Δ
ruins/apps/uncertainty.py 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06ad9d7...c543755. Read the comment docs.

@cojacoo
Copy link
Copy Markdown
Collaborator

cojacoo commented Apr 22, 2022

Very nice. I can easily hook into this, because the fun starts when the distributions are skewed and when there are some threshold processes changing your expected outcome. I will (so time permits) use your streamlit app to develop it further. However: I have an issue with git due to the LFS quota. If I clone the repo, it does not allow me to get all the branches... I would be happy to avoid any conflicting versions...

Copy link
Copy Markdown
Collaborator

@cojacoo cojacoo left a comment

Choose a reason for hiding this comment

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

Perfect basis. Thanks.

@mmaelicke
Copy link
Copy Markdown
Member Author

Skewed distributions are interesting. Then, I will make the used PDF function a changeable parameter for future developments. As of now, it is hard-coded to scipy.stats.norm.pdf. In the future, we can dynamically change it to any PDF supported by scipy. There are dozens of continuous distributions supporting the .pdf method: https://docs.scipy.org/doc/scipy/reference/stats.html#continuous-distributions

We can basically use any of them. We just need to keep in mind that they don't come with the same function signature, but I think we can limit ourselves to maybe 3 or 4 PDFs.

@mmaelicke mmaelicke merged commit 5a12e06 into main Apr 22, 2022
@mmaelicke mmaelicke deleted the uncertainty_draft branch April 22, 2022 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

uncertainty explorer Anything related to the uncertainty app

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

2 participants