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

Merge master into v1.5 #362

Merged
merged 333 commits into from Jan 23, 2019
Merged

Merge master into v1.5 #362

merged 333 commits into from Jan 23, 2019

Conversation

achilleas-k
Copy link
Member

This PR merges master into v1.5 with all the history and changes that came from the dev branch.

Changes made to the info.json (version bumps) are preserved.

We need this merge to easily bring the release branch up to date with master when preparing releases.

Accidentally added when merging changes from master branch.
The removal of the bindings lets us clean out the Mixins so we can
rewrite the class in a much cleaner fashion. The File.open() function
now simply calls the File() constructor.
ProxyLists are going to be removed but for now, I need to duplicate this
definition to avoid an import cycle.
Was only supported in HDF5 backend.
- Dimension
- EntityWithSources
- Property
- Section
HDF5 object wrappers (h5group and h5dataset) submodule is now called
HDF5.
Top level files should retain original dates:
2015 for group.py and 2014 for the rest.
Travis issue causes packages to be built from source, which times out.
HDF5 object wrappers (h5group and h5dataset) submodule is now called
HDF5.
More efficient versions of ProxyList and RefProxyList
hkchekc and others added 27 commits January 17, 2019 14:34
… err for len(col) not correct/ del col_idx for unit.setter
* fixed broken links eddited install instruction to install nixio no bindings

* deleted all NIX c++ dependencies and information, added new "advance install"

* added mising link to NIXPy

* added beta version install instructions, fixed stable version install instruction
* small fix to append_column/ short col too long

* write_df_compat sole test

* fix bug for np.string not converted to vlen_str

* Revert "write_df_compat sole test" in this branch

This reverts commit a1c0ab1.
When looking for NIX to compile cross-compatibility tests, search both
in /usr and /usr/local directories, as well as any directories supplied
by the user through the env vars (NIX_LIBDIR, NIX_INCDIR, BOOST_LIBDIR,
BOOST_INCDIR)
Old way is deprecated and produces warnings.
Now building in conftest and sending bindir to tests via fixture. Tests
are disabled by being marked with 'skip' if the build fails and
force-compat is not specified.
The compatibility tests are no longer run by default when NIX is
detected. They can be run by specifying --nix-compat. If specified, the
tests are compiled and any errors will cause failure of the
compatibility tests. If the option is not specified, the tests are
skipped, even if NIX is available.

One reason for this change is that the compatibility tests were always
compiled when NIX was available, even when they are unselected, and
compilation slows down the tests.

With this change, they are only compiled and run when explicitly
enabled.
Rebasing of no-bindings-dev branch off master had leftover files that
were modified in master after the merge-base. Removing them here.
Counterpart to 779ee10 from before 1.5
dev rebase.
Counterpart to cca977c from before 1.5
dev rebase.
Bindings removal and internals rewrite: Merge to master
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.2%) to 89.71% when pulling 8dd3096 on achilleas-k:v1.5 into 3e1b850 on G-Node:v1.5.

Copy link
Member

@jgrewe jgrewe left a comment

Choose a reason for hiding this comment

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

since all of the changes have been approved before, I dare to approve it :)

@jgrewe jgrewe merged commit 0f5c27a into G-Node:v1.5 Jan 23, 2019
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.

None yet

6 participants