Skip to content

Backends #16

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
Merged
merged 34 commits into from
Aug 8, 2022
Merged

Backends #16

merged 34 commits into from
Aug 8, 2022

Conversation

@carloalbertobarbano
Copy link
Member

@carloalbertobarbano carloalbertobarbano commented Aug 8, 2022

Add support for multiple backends (numpy, torch, tensorflow), discussed in #11

andreped and others added 30 commits August 5, 2022 11:09
@andreped
started making submodules for tf and torch
89eabfb
@andreped
made subpackages
631e3ce
@andreped
updated tests to work with new structure
a1ea833
@andreped
keep imports in each subpackage separate
537a6aa
@andreped
separation complete, linking with inits work
a520b10
@andreped
bug fix in tests, updated example
3af9479
@andreped
test if backend install selection works with CI
e3e6493
@andreped
major rework on backend logic - seems to work locally
9de07ec
@andreped
changed pip install in CI
7eef272
@andreped
find-links not compatible with [] pattern
ae01195
@andreped
typo T not T1 lambda, [] pip stuff
e8baefe
@andreped
from module in test
23ef481
@andreped
debugging - forgot to wrap test in def?
20556a2
@andreped
reduced python version for tf test
d0c7f6a
@carloalbertobarbano
Merge pull request #13 from andreped/backends
4448536 
Proper backend support
@carloalbertobarbano
remove unnecessary import
cd9d1cc
@carloalbertobarbano
remove unnecessary lines
85e0d32
@carloalbertobarbano
add numpy module
8d1438a
@carloalbertobarbano
move numpy_macenko_normalizer to numpy module
05500a9
@carloalbertobarbano
simplify normalizers filename
45f3df2
@andreped
fix for simplifying imports, some refactoring
0f2a5c9
@andreped
Updated README given proper TF support
26b2f17
@andreped
renamed 'base' to 'core'. Refactored
ac632d7
@carloalbertobarbano
refactor imports
29dc815
@carloalbertobarbano
update tests
bf00cfc
@carloalbertobarbano
keep base package (renamed from core); resolve conflicts
2e5d8cc
@carloalbertobarbano
Merge branch 'andreped-backends' into backends
4753b7d
@carloalbertobarbano
rename torch_backend and tf_backend to torch and tf
2de9b5e
@carloalbertobarbano
fix import typo
9aa9021
@carloalbertobarbano
fix class path
347ebef
@carloalbertobarbano carloalbertobarbano merged commit e044373 into development Aug 8, 2022
@carloalbertobarbano carloalbertobarbano mentioned this pull request Aug 8, 2022
Closed
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.

3 participants

@carloalbertobarbano @andreped