Skip to content

Commit 8853256

Browse files
authored
Merge pull request #225 from pycroscopy/fitter_refactored
Fitter refactored, and the TODO items have all been done, although generation of a fit dataset is to be done externally for now using the internal reconstruct_function() method - simply use this with the existing metadata on a fit object to derive the fitted dataset. Still need to make a notebook to show how to use all of the features of the new fitter class.
2 parents a289a80 + d2e4a60 commit 8853256

File tree

8 files changed

+1083
-5
lines changed

8 files changed

+1083
-5
lines changed

.github/workflows/actions.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@ jobs:
3434
sudo apt-get update -qq
3535
python -m pip install --upgrade pip
3636
python -m pip install flake8 pytest coveralls
37+
python -m pip install ".[test]"
38+
python -m pip show SciFiReaders
3739
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
3840
3941
- name: install package

extras_require.txt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,4 @@
1-
mpi4py
1+
extras_require={
2+
'MPI': ["mpi4py"],
3+
'test': ["pytest", "pytest-cov", "SciFiReaders"],
4+
},

setup.cfg

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,10 @@ universal=1
1010
test=pytest
1111

1212
[tool:pytest]
13-
testpaths = tests docs
13+
testpaths = tests docs
14+
15+
[options.extras_require]
16+
test =
17+
pytest
18+
pytest-cov
19+
SciFiReaders

setup.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
'toolz', # dask installation failing without this
1616
'cytoolz', # dask installation failing without this
1717
'dask',
18+
'dask-ml',
1819
'h5py>=2.6.0',
1920
'matplotlib>=2.0.0',
2021
'distributed>=2.0.0',
@@ -60,7 +61,7 @@
6061
author_email='pycroscopy@gmail.com',
6162
install_requires=requirements,
6263

63-
tests_require=['pytest'],
64+
tests_require=['pytest', 'SciFiReaders'],
6465
platforms=['Linux', 'Mac OSX', 'Windows 11/10'],
6566
# package_data={'sample':['dataset_1.dat']}
6667
test_suite='pytest',
@@ -69,7 +70,7 @@
6970
include_package_data=True,
7071
# https://setuptools.readthedocs.io/en/latest/setuptools.html#declaring-dependencies
7172
extras_require={
72-
'MPI': ["mpi4py"]
73+
'MPI': ["mpi4py"], 'test': ["pytest", "pytest-cov", "SciFiReaders"]
7374
},
7475
# If there are data files included in your packages that need to be
7576
# installed, specify them here. If using Python 2.6 or less, then these

sidpy/proc/__init__.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,6 @@
33
"""
44

55
from . import fitter
6+
from . import fitter_refactor
67

7-
__all__ = ['fitter']
8+
__all__ = ['fitter', 'fitter_refactor']

0 commit comments

Comments
 (0)