Move internal signac files into .signac subdirectory #708
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## schema2 #708 +/- ##
===========================================
+ Coverage 86.24% 86.25% +0.01%
===========================================
Files 52 52
Lines 5037 5041 +4
Branches 1099 1100 +1
===========================================
+ Hits 4344 4348 +4
Misses 491 491
Partials 202 202
Continue to review full report at Codecov.
|
bdice
requested changes
Mar 4, 2022
bdice
approved these changes
Mar 6, 2022
vyasr
added a commit
that referenced
this pull request
Mar 14, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
vyasr
added a commit
that referenced
this pull request
Apr 14, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
vyasr
added a commit
that referenced
this pull request
Apr 19, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
vyasr
added a commit
that referenced
this pull request
Apr 19, 2022
* Initial migration to schema version 2 including config rename (#678) * Implement initial migration to schema version 2. * Require project-local config to be signac.rc (not .signacrc) and make searches stricter to match. * Standardize method for getting project config at a root. * Move config to .signac/config. * Fix import order. * Address PR comments. * Remove some unnecessary code. * Address final PR coments. * Use integer schema version numbers (#688) * Change schema versioning to use integer strings. * Switch from int strings to pure ints. * Update signac/contrib/migration/__init__.py Co-authored-by: Bradley Dice <[email protected]> Co-authored-by: Bradley Dice <[email protected]> * Remove project name from schema (#684) * Remove project id API. * Remove project name from config as part of migration. * Fix issues with config CLI and remove project from default cfg. * Address PR comments. * Change the str of a project to the str of its root directory. * Change Project constructor to use root directory (#706) * Change project constructor to accept a root directory instead of a config file. * Change Project repr. * Address easy PR comments. * Move internal signac files into .signac subdirectory (#708) * Move shell history. * Move sp_cache file. * Address PR comments. * Move discovery to separate functions. (#711) * Move discovery to separate functions. * Address first round of PR comments. * Address PR comments. * Apply suggestions. * Remove configurable workspace directory (#714) * Remove workspace configurability. * Implement workspace_dir migration. * Apply suggestions from code review Co-authored-by: Bradley Dice <[email protected]> * Address remaining PR comments. * Update tests/test_project.py * Remove mention of configurability from project workspace docstring * Address PR comments. Co-authored-by: Bradley Dice <[email protected]> Co-authored-by: Corwin Kerr <[email protected]> * Update description of schema migration. * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci Co-authored-by: Bradley Dice <[email protected]> Co-authored-by: Corwin Kerr <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
vyasr
added a commit
that referenced
this pull request
Apr 21, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
vyasr
added a commit
that referenced
this pull request
May 2, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
bdice
pushed a commit
that referenced
this pull request
Jun 14, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
bdice
pushed a commit
that referenced
this pull request
Aug 1, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
bdice
pushed a commit
that referenced
this pull request
Oct 7, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
bdice
pushed a commit
that referenced
this pull request
Oct 27, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
vyasr
added a commit
that referenced
this pull request
Oct 30, 2022
* Move shell history. * Move sp_cache file. * Address PR comments.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR moves the shell history and state point cache files into the .signac directory.
Motivation and Context
Resolves #197. I have intentionally chosen to do nothing with
signac_statepoints.jsonsince that file would only have been created by users actually calling thewrite_statepointsfunction and therefore I think we should leave it to users to delete that file if it is present.Types of Changes
1The change breaks (or has the potential to break) existing functionality.
Checklist:
If necessary: