-
Notifications
You must be signed in to change notification settings - Fork 38
#944 longitudinal normalization #958
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
Merged
+546
−178
Merged
Changes from 5 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
0623b9b
changed _scale_func_group to also work with 3D objects, updated doc c…
agerardy 9ef4818
added 3d object fixture to conftest
agerardy 4d5873c
attempted 3D version of scale_norm
agerardy 0247e5a
test for scale_norm
agerardy f579bc6
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 7fd87aa
fixed scale_norm and added 3D functionality and tests for minmax_norm…
agerardy a6117ad
Merge branch '944-longitudinal-normalization' of github.com:theislab/…
agerardy 07d187c
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 2260102
added 3D functionality and tests for all normalization functions. rem…
agerardy fefda75
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 799e84f
updated normalization to correctly work with selected variables. adde…
agerardy 654d355
Merge branch 'main' into 944-longitudinal-normalization
agerardy e7887d8
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 155a72f
fixed small 3d fixture not returning anything
agerardy 0efca8a
minor comment edits
agerardy cbb07d6
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] cd26ac2
Changed logic to work with layers and R as just a layer
agerardy 1f3c554
removed unnecessary comments and a nonfunctional test
agerardy f7e6bfa
3D normalization tests now work with edata_blobs_timeseries_small and…
agerardy da9cc22
removed unmecessary copy and fixed docstrings
agerardy 82bd238
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] fa7becc
added type parameters to new functions and removed old comments
agerardy e8808a7
Merge branch 'main' into 944-longitudinal-normalization
Zethson 81e4873
refined docstrings in test_normalization to be more informative
agerardy e38bc83
update examples for test functions
agerardy 0566a17
Merge branch 'main' into 944-longitudinal-normalization
agerardy 32d3869
Merge branch '944-longitudinal-normalization' of github.com:theislab/…
agerardy 312ec7a
Merge remote-tracking branch 'origin/main' into 944-longitudinal-norm…
agerardy bed0dd4
updated normalization functions and tests to work with layers instead…
agerardy 5d3381f
combined most tests into a basic test_norm_3D and test_norm_3D_precis…
agerardy f1b2e94
updated examples to use layers. numbers are not real examples anymore…
agerardy 89a14ca
fixed unnecessary copy(), uses array_not_implemented error, import DE…
agerardy 4416c3b
redid examples with real data and different layer name
agerardy a0a494a
split up precise tests, removed unnecessary shape checks, added comme…
agerardy 882bfc2
group wise normalization now works with 3D and with dask arrays. fixe…
agerardy faa8ef5
simplified test_norm_3D to only test basic functions. removed NotImpl…
agerardy b7c06ce
removed group wise normalization for dask arrays, raise notImplemente…
agerardy 028456a
forgot to tupdate some tests with NotImplementedError
agerardy f7af4ec
removed redundant assignments and use _offset_negative_values instead…
agerardy b8de4a9
added registered type to .register and cast to float32 instead of 64 …
agerardy 1d54535
made test_norm_power_integers more lenient to allow casts to float32 …
agerardy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.