-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Annotations extras #13228
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
Merged
Merged
Annotations extras #13228
Changes from 24 commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
054ad12
Add check
PierreGtch ee92613
Add details attribute to annotations
PierreGtch fdceb5b
Update tests
PierreGtch a421357
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 078d837
fix coquille
PierreGtch 68fb8b3
Update change log
PierreGtch 33a35a5
Merge branch 'annotations-details' of https://github.com/PierreGtch/m…
PierreGtch ecac5e2
Make DetailsDict private and add docstring
PierreGtch ec34833
Fix test
PierreGtch 7b1d70b
ise int_like instead of int
PierreGtch fe6e4a1
Apply suggestions from code review
PierreGtch 9bcbc3f
Remove _check_forbidden_values
PierreGtch f8999b2
Remove _check_forbidden_values
PierreGtch e8f6728
Rename details to extras
PierreGtch 09bb125
Fix _AnnotationsExtrasDict and add test
PierreGtch 32641c0
fix writers
PierreGtch 76da62a
Improve type description in docstrings
PierreGtch 03e1b5a
only have a list of dict internally (no None)
PierreGtch dfa428f
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 7730c2f
Update test
PierreGtch 507d7f7
Merge branch 'annotations-details' of https://github.com/PierreGtch/m…
PierreGtch 9b43610
Add missing docstring
PierreGtch 09dd2af
Merge branch 'main' into annotations-details
PierreGtch dceb489
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b2c89e9
Apply review suggestion
PierreGtch 3f2003a
Fix tests
PierreGtch 1d2b30c
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 1b2b05d
Fix docstrings
PierreGtch a94dc4f
Apply suggestions from code review
PierreGtch ab0f067
Improve `test_extras_dict_raises`
PierreGtch af8add6
Make extras_columns private
PierreGtch f924623
Remove extras_data_frame attribute
PierreGtch 088858d
Support saving to csv and txt
PierreGtch 74a480a
simplify assert
PierreGtch 2f5a04e
Simplify read txt
PierreGtch 94c06a9
Merge branch 'main' into annotations-details
PierreGtch f2a6b76
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 061eb58
pre-commit
PierreGtch d948ca8
Add _AnnotationsExtrasList container
PierreGtch d5e823c
simplify read CSV
PierreGtch ad0baa0
Update test
PierreGtch 9e55e70
Warn when writing heterogeneous csv extras
PierreGtch cd9f08f
Warn when writing heterogenous dtypes in txt
PierreGtch 42447bc
test warnings
PierreGtch 960a7c2
Fix annotations list repr
PierreGtch 1e58564
Infer TXT types using pandas, if possible
PierreGtch 28448ae
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 50f5385
Fix docstring format
PierreGtch 77f35ba
Merge branch 'main' into annotations-details
PierreGtch cd716b9
Fix docstring format
PierreGtch ac07fa5
codespell
PierreGtch 09b43ca
Remove unused function
PierreGtch 445bf01
Skip mypy check
PierreGtch 6de0f4c
Apply review suggestion (spelling)
PierreGtch 3ea3a9a
Merge branch 'main' into annotations-details
larsoner 9e67e3b
Apply suggestions from code review [circle full]
PierreGtch 4acce8d
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 3621595
[circle full]
drammock 7406948
Fix positional arguments [circle full]
PierreGtch 8af93a2
Add test for positional args
PierreGtch e7aa632
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 1668571
Add missing docstring
PierreGtch e908e89
Merge branch 'main' into annotations-details
PierreGtch efff25a
revert new test (unneeded)
drammock File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Add a ``details`` attribute to :class:`mne.Annotations`, by `Pierre Guetschel`_. | ||
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.