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.

Sign up for GitHub

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

Jump to bottom

Closes #4 - wrapper around some cleaning steps #6

Merged
merged 25 commits into from
Apr 10, 2020
Merged

Closes #4 - wrapper around some cleaning steps #6

merged 25 commits into from
Apr 10, 2020

Conversation

erkandem
Copy link
Owner

  • fixed typos
  • requirements relaxes
  • pypy3 dropped in CI
  • takes 12 minutes, and constraints scipy version -> find a better solution
  • py3.8 added in CI
  • bugfix (slicing and file paths in example3)
  • ...

erkandem added 25 commits April 9, 2020 14:50
@erkandem
update readme
5a5f2cf 
 - update and correct email address
 - remove link to site
@erkandem
update email address in setup.py
44563b9
@erkandem
fix typo
c115c92
@erkandem
clean up
71f4de1 
 - commented code
 - unused statement
 - unneeded/confusing comments
@erkandem
remove unused imports
c6fc0f1
@erkandem
remove usage of io
e9d87c0
@erkandem
apply *don't use abbreviations* to ex -> example
25b5172
@erkandem
apply *don't use abbreviations* to expect_ex_1 -> `expected_result_…
60fc65b 
…example_1`
@erkandem
narrow down exception
5622d15
@erkandem
fix typo
fb1a3f6
@erkandem
narrow down exception and fix typo
9e51e56
@erkandem
one way of fencing/quoting
ade8889
@erkandem
don't need to initialize Error
69dbc8b
@erkandem
adjust code to work with zipped data
45b3b5b 
 - 2.1MB -> .4MB
@erkandem
made utility functions importable
6def8b4
@erkandem
basic tests for some utilities. hypothesis package might be better …
70f0460 
…suited than `pytest.mark.parametrize`
@erkandem
order import switch to more usual import numpy as np. same for `pyt…
588a979 
…est`

remove trailing comma at assertion
@erkandem
python depends on which version it is linked to. changed to `python…
9b6b511 
…3` since that `python2` isn't covered anyway
@erkandem
CI change
2275581 
 - drop pypy support rather than limit `scipy` version
@erkandem
requirements change
5e72679 
 - since a numpy version 1.15 specific feature isn't used anymore the constraint may be lifted
@erkandem
remove pypy support from setup.py
d619b7c
@erkandem
try to add python 3.8 to CI
cc25a35
@erkandem
slicing error not noticed because try except clause caught everything
228bec2 
convert to np.ndarray in order to slice with and np.ndarray of booleans
@erkandem
typo in comment
036ec6f
@erkandem
add 3.8 to shield
2093bfc
@erkandem erkandem changed the title Issue 4 20200408 Closes #4 - wrapper around some cleaning steps Apr 10, 2020
@erkandem erkandem merged commit 400060d into master Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@erkandem