Skip to content

Comments

Add implementation of standard deviation on data#1

Open
C-L-Grove wants to merge 1 commit intomainfrom
feature-std-dev
Open

Add implementation of standard deviation on data#1
C-L-Grove wants to merge 1 commit intomainfrom
feature-std-dev

Conversation

@C-L-Grove
Copy link
Owner

@C-L-Grove C-L-Grove commented Jun 20, 2024

Opening a pull request

Copy link

@tgermany20 tgermany20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment


def s_dev(data):
"""Computes and returns standard deviation for data."""
mmm = np.mean(data, axis=0)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, no changes from me. Maybe more comments in code

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah will do

for filename in InFiles:
inflammation_data = models.load_csv(filename)

view_data = {'average': models.daily_mean(inflammation_data), 'max': models.daily_max(inflammation_data), 'min': models.daily_min(inflammation_data)}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possible test to complete, test standard deviation for one patient with multiple observations.

@@ -27,7 +27,8 @@ def main(args):
for filename in InFiles:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test standard deviation for two patients.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brilliant, I'll raise this as an issue on my repo for future work when I work out how.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants