bryanhanson / LearnPCA

Functions, Data Sets and Vignettes to Aid in Learning Principal Components Analysis (PCA)
https://bryanhanson.github.io/LearnPCA
10 stars 4 forks source link

Draft Step_By_Step_PCA Vignette #3

Closed bryanhanson closed 3 years ago

bryanhanson commented 3 years ago

@dtharvey I fixed some errors and extended the Step_By_Step vignette; can you take a look at Step 4 and below?

dtharvey commented 3 years ago

Will do tomorrow. Left my computer at school!

On Mon, Oct 18, 2021 at 6:14 PM Bryan Hanson @.***> wrote:

@dtharvey https://github.com/dtharvey I fixed some errors and extended the Step_By_Step vignette; can you take a look at Step 4 and below?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bryanhanson/LearnPCA/pull/3#issuecomment-946210315, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXZG6GHMWBYENEZCN7CGSLUHSL3NANCNFSM5ASJTQWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- Sent from my iPhone.

dtharvey commented 3 years ago

Hi Bryan

Took me awhile to find the git cheatsheet your prepared for me. I managed to update the LearnPCA folder that sits on my desktop. I found the step-by-step vignette, but when I tried to knit it into html I got the following error message:

processing file: Step_By_Step_PCA.RmdQuitting from lines 30-43 (Step_By_Step_PCA.Rmd) Error in library("LearnPCA") : there is no package called 'LearnPCA' Calls: ... suppressPackageStartupMessages -> withCallingHandlers -> library Execution halted

It is choking on the line suppressPackageStartupMessages(library("LearnPCA")) in the first code block.

Any idea what is going on here?

Take care, David

On Mon, Oct 18, 2021 at 6:14 PM Bryan Hanson @.***> wrote:

@dtharvey https://github.com/dtharvey I fixed some errors and extended the Step_By_Step vignette; can you take a look at Step 4 and below?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bryanhanson/LearnPCA/pull/3#issuecomment-946210315, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXZG6GHMWBYENEZCN7CGSLUHSL3NANCNFSM5ASJTQWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- David Harvey Professor of Chemistry & Biochemistry DePauw University @.***

bryanhanson commented 3 years ago

That sounds like LearnPCA is not installed on your computer. Did you recently upgrade your R version? That would do it. Regardless, you need to build, check and install LearnPCA locally. See if that fixes it.

dtharvey commented 3 years ago

Ah! That is possible. I’ll give that a try.

On Tue, Oct 19, 2021 at 5:53 PM Bryan Hanson @.***> wrote:

That sounds like LearnPCA is not installed on your computer. Did you recently upgrade your R version? That would do it. Regardless, you need to build, check and install LearnPCA locally. See if that fixes it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bryanhanson/LearnPCA/pull/3#issuecomment-947134603, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXZG6BJHG4MONT336QP7ALUHXSHDANCNFSM5ASJTQWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- Sent from my iPhone.

dtharvey commented 3 years ago

Hi Bryan

I have LearnPCA open in RStudio as a project, which provides Git access. I've pasted in an image of what I see. Alas, I am not clear on what I do from here to pass along suggested edits to the Step-by-Step PCA Rmd file. Do I just make suggested edits to the file, save locally, commit, and then push? I understand that edits can be made directly to the text and/or code; but, what is the best way to pass along a comment where making edits does not make sense?

Getting a booster vaccine shot this afternoon; my birthday in September moved me up into the 65+ demographic.

Hope all of you are doing well!

Take care, David

[image: Screen Shot 2021-10-22 at 08.51.44 AM.png]

On Tue, Oct 19, 2021 at 5:53 PM Bryan Hanson @.***> wrote:

That sounds like LearnPCA is not installed on your computer. Did you recently upgrade your R version? That would do it. Regardless, you need to build, check and install LearnPCA locally. See if that fixes it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bryanhanson/LearnPCA/pull/3#issuecomment-947134603, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXZG6BJHG4MONT336QP7ALUHXSHDANCNFSM5ASJTQWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

-- David Harvey Professor of Chemistry & Biochemistry DePauw University @.***

bryanhanson commented 3 years ago

Simply a comment? To keep it out of the actual work, I guess you could reference it by line number and make it here, though the line number would only be accurate until we change it again. Or put it in the document in all caps or with your initials and I will find it and eventually remove it.

EDIT: I could not see the screenshot so suggestion above is general.

bryanhanson commented 3 years ago

Merging; further work will be needed but this is a good point to merge it.