-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Description
List of things that need to be renamed:
-
filters.py/get_freq_spectrum_notdict
- incorporate this as boolean option inget_freq_spectrum
, and rename toget_frequency_spectrum
-
permute.py/reorder_connectome
- this function seems oddly specific to desikan-killiany ordering, maybe rename to something related to DK. - there's a replicate
get_freq_spectrum
inpreprocess.py
- maybe it's worthwhile to rename
preprocess/preprocess.py
- we should consider removing
read/data_reader.py
all together, they are too specific for our use case -
eigenmode.py
needs an overhaul, @axiezai -
laplacian.py
needs renames for functions that gets complex/regular and one specific function for spectral graph model that needs frequency as an input -
network_transfer.py
has 2 network transfer functions, need to do the same as laplacian for the alpha parameters - run
pylint
after everything is done
Metadata
Metadata
Assignees
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed