Skip to content

Path density for alanine dipeptide TPS example#511

Merged
jhprinz merged 23 commits into
openpathsampling:masterfrom
dwhswenson:path_density
Jul 7, 2016
Merged

Path density for alanine dipeptide TPS example#511
jhprinz merged 23 commits into
openpathsampling:masterfrom
dwhswenson:path_density

Conversation

@dwhswenson

@dwhswenson dwhswenson commented Jul 1, 2016

Copy link
Copy Markdown
Member

Fixes some problems in the path density histogram and adds a plot to the example notebook

  • Fix path histogram for sequential points in the same cell
  • Test for sequential points in same cell in path histogram
  • Simplify plotting of path density histograms
  • Add PathDensityHistogram to alanine_dipeptide_tps_analysis.ipynb
  • Tests and docstrings for plotting

@dwhswenson dwhswenson added this to the 1.0 milestone Jul 1, 2016
@dwhswenson dwhswenson self-assigned this Jul 1, 2016
@dwhswenson dwhswenson changed the title [WIP] Path density for alanine dipeptide TPS example Path density for alanine dipeptide TPS example Jul 5, 2016
@dwhswenson

Copy link
Copy Markdown
Member Author

Ready for review and merge! Left the outputs on the AD_tps_analysis clean, rather than cause more errors. But you can see the path densities in earlier versions, and I'll be adding one to the paper soon.

@dwhswenson dwhswenson assigned jhprinz and unassigned dwhswenson Jul 5, 2016
@jhprinz

jhprinz commented Jul 7, 2016

Copy link
Copy Markdown
Contributor

The notebooks look good. Not sure if I completely got the HistogramPlotter, but seems to work as expected. Will merge.

@jhprinz jhprinz merged commit 28c7991 into openpathsampling:master Jul 7, 2016
@dwhswenson dwhswenson deleted the path_density branch March 15, 2018 14:22
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.

2 participants