Hacky modularization example#28
Closed
leej3 wants to merge 27 commits into
Closed
Conversation
Commiting first copy of pycharm (.py) python code file that I have been working on during the past month to read EEG files and begin preliminary exploration.
…ng with date and time information, to compare with values from biometric insturments such as EEG and GSR. Also some initial coding in the main python file for reading and visualizing this information.
…pearson r for variables that would like to explore in Mood_Focus_Table from Excel
…me feedback on related to how to graph the input from a specific sensor. I am also adding the EEG file that is being analysed
…ich can do pearson correlation analysis on two columns of data such as mood and alpha amplitude, and also adding the append_table_function which can append this same overarching spreadsheet of subjective and biometric data from new records, updating it with new relevant entries/recordings.
… more concise and clear comments to guide reader through each function
… good readability by user
q Merge branch 'master' of https://github.com/barlehmann/EEG_GSR_Wellness_Tracker i 'bad files' q
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This is a quick summary of our discussion regarding #18
You can close this, you shouldn't merge it. I just wanted to show you the general idea of functions I showed you. You would have to add docstrings, and correctly pass arguments in etc. but modularizing like this is better. You can iterate in a notebook and then move the functions to modules once you are happy with them.
The name... approach will allow you to create executable scripts that you can still import from (though you could instead just fully separate files you run as scripts and ones from which you import).
Also note the directory your package's python files should be sitting in...