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

overplotting in plotParameterSamples - Parameter Distribution 2D #44

Open
FFroehlich opened this issue Feb 14, 2017 · 6 comments
Open
Assignees

Comments

@FFroehlich
Copy link
Collaborator

plotting 2D parameter distributions with >10^5 samples (nsimu_run>10^6) takes long.
I think we should limit the maximum number of points to a reasonable number and use mvksdensity to color the plots by density

@paulstapor paulstapor self-assigned this Feb 26, 2017
@BBallnus
Copy link
Contributor

BBallnus commented Mar 3, 2017

The reason why it takes so long is probably because each point is plotted separately and thus adding up the matlab figure overhead each time. Plotting all points at once speeds up the process a lot. Combining this with an plotting-internal thinning to a maximum point number should fix the problem.

Using density plots might be a nice option to scatter but I would not replace the "classic scatter" entirely since it provides a clearer view onto the non-processed results.

@FFroehlich
Copy link
Collaborator Author

I agree. But I did not suggest replacing the scatter plot, but instead to color the points of the scatterplot according to density (like this: http://mathematica.stackexchange.com/questions/85530/how-2d-scatterplots-with-quantitative-density-dependent-coloring). Would be an option to only activate this when a threshold of points is exceeded and the computed density is somewhat reasonable.

@paulstapor
Copy link
Collaborator

Holy crap, that's beautiful! :D

@FFroehlich
Copy link
Collaborator Author

the only matlab routine I know that does density scatters is https://de.mathworks.com/matlabcentral/fileexchange/8430-flow-cytometry-data-reader-and-visualization/content/dscatter.m but it is quite buggy. Yet, it really is only 2-3 lines of codes to implement this with mvksdensity (or better make an independent implementation that does not rely on the statistics toolbox).

@BBallnus
Copy link
Contributor

BBallnus commented Mar 3, 2017

fancy indeed!

@paulstapor
Copy link
Collaborator

I like the whole blueish thing plot nr 2, since it does process the data too much. The other pictures on the website are maybe a bit too much, although it might be an option thinking of some additional color coding for those regions which are completely filled with samples, to get even more information about the density. But here, things start to be very tricky...

@dweindl dweindl added this to the Release 1.0 milestone Mar 10, 2017
@paulstapor paulstapor modified the milestones: Release 1.0, CleanUp + some new Feature (v1.0.1) Sep 18, 2017
@paulstapor paulstapor modified the milestones: CleanUp + some new Feature (v1.0.1), Restructure figure generation and plotting routines Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants