-
Notifications
You must be signed in to change notification settings - Fork 26
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
add multiprocessing #92
Open
LFT18
wants to merge
99
commits into
RasmussenLab:developer
Choose a base branch
from
LFT18:main
base: developer
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
0e2242e
Add files for multiprocessing
2add9b9
Update identify_associations_multiprocess.py
LFT18 f645ea4
Clean multiprocessing script
LFT18 f471704
Update __main__.py multiprocessing
LFT18 85c28e5
Update schema.py multiprocessing
LFT18 6330e92
Update __init__.py multiprocessing
LFT18 bbe1b4e
Update preprocessing.py
LFT18 820c554
:fire: clean-up duplicated src/move files (pkg was in main folder)
ce9a9dc
:sparkles: add identify_associations_multiprocess to src/move/tasks
5327223
:bug: make mutliprocessing not stale: assign # of threads for each pr…
eaa858a
Merge pull request #1 from enryH/main
LFT18 5ab5e59
Updated identify_associations_multiprocess.py
33f565a
Update config files for small tries
63f128b
Multiprocessing for analyze_latent
ca389d2
Analyze latent multiprocessing
e08a94b
Analyze latent multiprocessing
e94ef90
Fix bayes_k calculation
f4f0aa3
Fix analyze_latent_multiprocessing
6a0b665
Update and new functions
a5310a6
Delete files and fix multiloop
e67bb75
Clean identify_association_multiprocess.py
86bfed5
Clean analyze_latent multiprocessing.py
f9d4961
Update perturbations.py
LFT18 c2c49e8
Update perturbations.py
LFT18 0a4bcae
Delete src/move/tasks/analyze_latent_efficient.py
LFT18 ce20dac
Delete src/move/tasks/analyze_latent_multiprocessing.py
LFT18 4a72842
Delete src/move/tasks/identify_associations_multiprocess_loop.py
LFT18 f537d21
Delete src/move/tasks/identify_associations_multiprocess_may.py
LFT18 568aaa8
Delete src/move/tasks/identify_associations_selected.py
LFT18 ede3707
Delete src/move/tasks/analyze_latent_original.py
LFT18 5df8a01
Remove multiprocess_loop
LFT18 52c37fd
Remove multiprocess_loop
LFT18 2e29f23
:art: format with black
13678cb
Merge branch 'main' into LFT18-main
f92a862
Merge branch 'developer' into LFT18-main
b7824f7
:art: add trigger of actions from PR
e5253a2
:art: format with black
d4118a3
:fire: remove duplicated code and intermediate scripts
3e19e24
Merge branch 'developer' into LFT18-main
dbc0238
:bug: fix f-string formatting errors
80352e6
:bug: remove unused imports
488b4a4
:rewind: add configuration files back in from developer branch
9b3a27e
:art: isort imports
05d4c34
:construction: see if this advances CI to the next step
ebb72ad
:fire: remove intermediate files of development
efbfd5c
:construction: multiprocess only defined for bayes factors
fbbeb19
:bug: remove non-existing, intermediate tasks (used for developing), …
cb3ad30
:bug: also deactivate mutliprocessing for KS as it's not implemented
8c61d35
:art: fix flake8-bugbear issues except missing multiprocessing of t-t…
5aa03ff
:bug: format and fix import
8b06298
:bug: use perturb_continuous_data_extended from perturbations
6cfd1f8
:fire: comments and old configurations; format
8277891
:fire: remove duplicated functionality
44802eb
:sparkles: integrate multiprocessing into analyze_latent.py
f64d779
:sparkles: merge multiprocessing bayes factors into identify_associat…
6a17110
:fire: remove old schema entries, increase run time
b8b4769
:zip: do no save intermediate files for single-process bayes_approach
2927d3f
:fire: remove comments
202eb74
:fire: remove unused code
d6bc896
:art: reorder functions
b99ce97
:construction: move bayes_parallel to own module
171c915
:construction: unify interface
95c9f40
:fire: remove not-used code
e556fae
:art: start separating recurrent code into fcts
1a2774d
:art: adapt to look more similar to single-core bayes factor fct
680c164
:sparkles: add back masking of self-perturbed feat.
8f43c90
:art: initiailize logger at the top of the module
6178c8f
:sparkles: pass feature_mask to bayes_parallell
60ed227
:bug: add condition for masking
f5da671
:art: align masking strategies
7eae82d
:bug: fix cont perturbation
50a623e
:bug: remove redefintion of nan_mask
2df057b
:art: only define logger once in module
2e87e12
:art: align single process bayes and multiprocess bayes fct
aa2e5d9
:art: just document in code that this cannot happen
4041fcb
:zap: improve CI speed, reduce stability (-> one refit only)
2d004e0
:bug: use default no. of epochs + t-test needs 4 refits
8d65528
:zap: do not run t-test check (for now)
1dd6788
:zap: bump up bayes factor training
dc9020e
:art: train both refits with 100 epochs
9cd2a7b
:sparkles: add log2 option
a4911d7
:art: document some more
e0421bd
:zap: test multiprocess on continuous tutorial
c70d328
:bug: remove non-exisitng key
1c72316
:sparkles: build dataloader fct
58f08e4
:bug: fix minor bug (wrongly assigned feat)
f895237
:zap: move masking code into main fct of module
5eb7954
:art: move feat_mask creation out
8c4e53b
:ambulance: temp. fix of CI
49a93d0
:zap: do not build dataloaders for multiprocessing
709c674
:construction: test t-test again, re-run pert. w/o model training
6e65cc6
:sparkles: add categorical pert. to multiprocessing
4efbdd9
:fire: remove unused code
dab767a
:art: remove unused argument
980bbce
:rewind: checkout developer version
c26b2dd
:art: move shared key to base class
05c1735
:fire: remove comments and code duplications
c5002cd
:art: update type hints, remove unused import
fe8c48b
Merge branch 'developer' into main
enryH 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
There are no files selected for viewing
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this correct? Or does it still need to be changed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's correct now. I cannot check right now because I've been having problems connecting to Esrum all morning, but I'll check as soon as it works again (hopefully soon)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still can't connect, it's very annoying :( . I'll let you know as soon as I can again, but I think the code should be fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to connect finally today at noon :). The file was correct, because those functions are not used at all for multiprocessing, I had just changed that to test some things with the previous functions. But it is true that it led to confusion, so I reverted the changes so that the not used functions have their original code