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

Rework DynamicTable/DynamicTableRegion.get when the table contains a DTR #579

Merged
merged 22 commits into from
Jul 5, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 14 additions & 17 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# HDMF Changelog

## HDMF 3.0.0 (Upcoming)
## HDMF 3.0.0 (July 3, 2021)

### New features
- Add support for Python 3.9, drop support for Python 3.6. @rly (#620)
Expand All @@ -15,26 +15,23 @@
`h5py.Dataset` object, and `d[:]` returns `str` objects. Under HDMF 3.x, the same dataset `d` is read
as a `hdmf.utils.StrDataset` object and `d[:]` still returns `str` objects.
- Add RRID to docs. @oruebel (#633)
-
- Allow passing ``index=True`` to ``DynamicTable.to_dataframe()`` to support returning `DynamicTableRegion`` columns as indices or Pandas DataFrame. @rly (#579)
- Updated external resources tutorial. @mavaylon (#611)

### Breaking changes and deprecations
-
-
-
-

### Internal improvements
-
-
-
-
- Previously, when using ``DynamicTable.__getitem__`` or ``DynamicTable.get`` to access a selection of a
``DynamicTable`` containing a ``DynamicTableRegion``, new columns with mangled names for the table data referred to
by the ``DynamicTableRegion`` were added to the returned DataFrame. This did not work properly for ragged
``DynamicTableRegion``, multiple levels of nesting, or multiple rows returned.
Now, these methods will by default return columns of indices of the ``DynamicTableRegion``. If ``index=False`` is
passed to ``DynamicTable.get``, then nested DataFrames will be returned, one DataFrame per row of the original
resulting DataFrame. @rly (#579)

### Bug fixes
- Update the validator to allow extensions to data types which only define data_type_inc @dsleiter (#609)
- Fix error when validating lazy-loaded datasets containing references @dsleiter (#609)
-
-
-
- Update the validator to allow extensions to data types which only define data_type_inc. @dsleiter (#609)
- Fix error when validating lazy-loaded datasets containing references. @dsleiter (#609)
- Fix error when using ``DynamicTable.__getitem__`` or ``DynamicTable.get`` when table has a ragged
``DynamicTableRegion``. @rly (#579)

## HDMF 2.5.8 (June 16, 2021)

Expand Down
2 changes: 1 addition & 1 deletion requirements-min.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# minimum versions of package dependencies for installing HDMF
h5py==2.9 # support for setting attrs to lists of utf-8 added in 2.9
h5py==2.10 # support for selection of datasets with list of indices added in 2.10
numpy==1.16
scipy==1.1
pandas==1.0.5
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
schema_dir = 'common/hdmf-common-schema/common'

reqs = [
'h5py>=2.9,<4',
'h5py>=2.10,<4',
'numpy>=1.16,<1.21',
'scipy>=1.1,<2',
'pandas>=1.0.5,<2',
Expand Down
Loading