Skip to content

Release 1.7.0#1160

Merged
dwhswenson merged 66 commits into
stablefrom
release-1.7.0
Oct 4, 2024
Merged

Release 1.7.0#1160
dwhswenson merged 66 commits into
stablefrom
release-1.7.0

Conversation

@dwhswenson

Copy link
Copy Markdown
Member

OpenPathSampling 1.7.0 contains several pieces of new functionality and some bug fixes. Most significant is probably the experimental new path tree functionality, which works with data stored from SimStore ("experimental" storage, now recommended for most use cases).

Enhancements

  • Add restart_at_step for ExternalEngine (for numeric filenames) (#1105)
  • Custom PathDensityHistogram._add_ops_trajectory (#1154 #refactor)
  • Volume shooting point selector (#1151)

Experimental (beta) features

  • SimStore performance improvements and fixes (#1157 #enhancement)
  • New path tree (#1048 #enhancement)

Bugs fixed

  • Fix SQLStorageBackend.load_storable_function_table (#1153 #experimental)
  • Fix histogram xvals (#1150)

Miscellaneous improvements

  • Let UUID determine equality for ensembles (#1155)
  • Improvements to continuous integration / automated testing (#1149, #1156)
  • Improvements to documentation (#1159)

dwhswenson and others added 30 commits August 12, 2021 19:51
This will need some cleanup, but it is in reasonable shape now
Might update the outputs from CLI help commands
This looks like a SQLAlchemy 2.0 issue that isn't covered in the current
unit tests, but is covered in some more extensive external tests.
This adds a touch point for path density histogram subclasses to change
what occurs when adding an OPS trajectory to a PathDensityHistogram.

I needed this when playing with the idea of having a custom histogram
that would also track all trajectories that passed through a given
histogram bin.
dwhswenson and others added 24 commits August 13, 2024 11:01
Let UUID determine equality for ensembles
Co-authored-by: Sander Roet <sanderroet@hotmail.com>
Co-authored-by: Sander Roet <sanderroet@hotmail.com>
SimStore performance improvements and fixes
This is required to correctly build the CLI docs.

If a user tries to build the docs without the CLI installed, that
should still be fine, but it won't get the CLI parameters.
Co-authored-by: Sander Roet <sanderroet@hotmail.com>
Add `restart_at_step` for `ExternalEngine` (for numeric filenames)
@codecov

codecov Bot commented Oct 2, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.32%. Comparing base (9131891) to head (2605e58).
Report is 177 commits behind head on stable.

Additional details and impacted files
@@            Coverage Diff             @@
##           stable    #1160      +/-   ##
==========================================
- Coverage   81.76%   81.32%   -0.45%     
==========================================
  Files         142      142              
  Lines       15612    15659      +47     
==========================================
- Hits        12765    12734      -31     
- Misses       2847     2925      +78     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dwhswenson

Copy link
Copy Markdown
Member Author

Merging this will create a new OPS release.

It looks to me like project coverage issues here might be related to deprecations that are no longer supported. I'm willing to allow that since patch coverage is 100%, and all other tests pass.

This is ready for review and comment. Without review, I will merge this after at least 24 hours, no sooner than Fri 04 Oct 01:00 GMT (Thu 03 Oct 20:00 my local).

@sroet sroet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@dwhswenson dwhswenson merged commit 857917c into stable Oct 4, 2024
@dwhswenson dwhswenson deleted the release-1.7.0 branch October 4, 2024 01:54
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